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

Flexible TupleDomain.intersect type #11078

Merged
merged 1 commit into from
Feb 23, 2022
Merged

Flexible TupleDomain.intersect type #11078

merged 1 commit into from
Feb 23, 2022

Conversation

findepi
Copy link
Member

@findepi findepi commented Feb 17, 2022

Allow TupleDomain.intersect to operate on objects with different (but
related) key types.

@findepi
Flexible TupleDomain.intersect type
5ffa85a 
Allow `TupleDomain.intersect` to operate on objects with different (but
related) key types.
@findepi findepi requested review from martint and losipiuk February 17, 2022 10:10
@cla-bot cla-bot bot added the cla-signed label Feb 17, 2022
This was referenced Feb 17, 2022
Merged
Closed
@martint
Copy link
Member

martint commented Feb 17, 2022

What’s the use case for this?

@findepi
Copy link
Member Author

findepi commented Feb 17, 2022

What’s the use case for this?

#10215 (comment)

@findepi findepi merged commit 484ee43 into trinodb:master Feb 23, 2022
@findepi findepi deleted the findepi/tupledomain-intersect-types branch February 23, 2022 10:54
@github-actions github-actions bot added this to the 372 milestone Feb 23, 2022
@mosabua mosabua mentioned this pull request Feb 23, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@losipiuk losipiuk losipiuk approved these changes

@martint martint Awaiting requested review from martint

Assignees
No one assigned
Labels
cla-signed
Milestone
372
Development

Successfully merging this pull request may close these issues.
3 participants
@findepi @martint @losipiuk