Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ux: reimplement the inner join convenience to not repeat fields in case of equality join predicates #7820

Closed
1 task done
kszucs opened this issue Dec 20, 2023 · 1 comment
Closed
1 task done

ux: reimplement the inner join convenience to not repeat fields in case of equality join predicates #7820

kszucs opened this issue Dec 20, 2023 · 1 comment
Labels
tes-required-for-merge Issues that must addressed before merging the-epic-split branch into main ux User experience related issues
Milestone
9.0

Comments

@kszucs
Copy link
Member

kszucs commented Dec 20, 2023

What happened?

Follow-up of #7752

What version of ibis are you using?

the-epic-split branch

What backend(s) are you using, if any?

No response

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@kszucs kszucs added the ux User experience related issues label Dec 20, 2023
@kszucs kszucs mentioned this issue Dec 20, 2023
Merged
@cpcloud cpcloud changed the title ux reimplement the inner join convenience to not repeat fields in case of equality join predicates. ux: reimplement the inner join convenience to not repeat fields in case of equality join predicates. Dec 21, 2023
@cpcloud cpcloud changed the title ux: reimplement the inner join convenience to not repeat fields in case of equality join predicates. ux: reimplement the inner join convenience to not repeat fields in case of equality join predicates Dec 21, 2023
@kszucs kszucs added the tes-required-for-release Things that must be addressed before *release* of `main` after merging in `the-epic-split` label Dec 26, 2023
@cpcloud cpcloud added the tes-required-for-merge Issues that must addressed before merging the-epic-split branch into main label Jan 8, 2024
@kszucs kszucs mentioned this issue Jan 29, 2024
Merged
@jcrist jcrist removed the tes-required-for-release Things that must be addressed before *release* of `main` after merging in `the-epic-split` label Jan 30, 2024
@cpcloud cpcloud added this to the 9.0 milestone Jan 31, 2024
@kszucs
Copy link
Member Author

kszucs commented Feb 3, 2024

Resolved by #8127

@kszucs kszucs closed this as completed Feb 3, 2024
@github-project-automation github-project-automation bot moved this from backlog to done in Ibis planning and roadmap Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tes-required-for-merge Issues that must addressed before merging the-epic-split branch into main ux User experience related issues
Projects
Ibis planning and roadmap
Archived in project
Milestone
9.0
Development

No branches or pull requests
3 participants
@cpcloud @kszucs @jcrist