Skip to content

Clean up mixed join common utilities#20836

Merged
rapids-bot[bot] merged 9 commits intorapidsai:mainfrom
PointKernel:join-cleanups
Jan 6, 2026
Merged

Clean up mixed join common utilities#20836
rapids-bot[bot] merged 9 commits intorapidsai:mainfrom
PointKernel:join-cleanups

Conversation

@PointKernel
Copy link
Copy Markdown
Member

@PointKernel PointKernel commented Dec 11, 2025

Description

Contributes to #19270

This is the first PR aimed at improving and tidying up the internal utilities used for joins. It focuses on mixed joins: several utilities specific to mixed joins were previously placed in the general join utilities header, and the mixed semi/anti join utilities were located in the standard mixed join header. This PR reorganizes those utilities into more appropriate locations and cleans up header inclusions across all join source files.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@PointKernel PointKernel self-assigned this Dec 11, 2025
@PointKernel PointKernel added libcudf Affects libcudf (C++/CUDA) code. improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Dec 11, 2025
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Dec 11, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@PointKernel
Copy link
Copy Markdown
Member Author

/ok to test cf2edc4

@PointKernel
Copy link
Copy Markdown
Member Author

/ok to test 890ff7d

@PointKernel PointKernel marked this pull request as ready for review December 12, 2025 20:49
@PointKernel PointKernel requested a review from a team as a code owner December 12, 2025 20:49
@PointKernel PointKernel added the 3 - Ready for Review Ready for review by team label Dec 12, 2025
Copy link
Copy Markdown
Contributor

@kingcrimsontianyu kingcrimsontianyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for the work!

@PointKernel
Copy link
Copy Markdown
Member Author

/merge

@rapids-bot rapids-bot bot merged commit cc714dc into rapidsai:main Jan 6, 2026
268 of 270 checks passed
@PointKernel PointKernel deleted the join-cleanups branch January 6, 2026 17:49
galipremsagar pushed a commit to galipremsagar/cudf that referenced this pull request Jan 9, 2026
Contributes to rapidsai#19270

This is the first PR aimed at improving and tidying up the internal utilities used for joins. It focuses on mixed joins: several utilities specific to mixed joins were previously placed in the general join utilities header, and the mixed semi/anti join utilities were located in the standard mixed join header. This PR reorganizes those utilities into more appropriate locations and cleans up header inclusions across all join source files.

Authors:
  - Yunsong Wang (https://github.com/PointKernel)

Approvers:
  - Tianyu Liu (https://github.com/kingcrimsontianyu)
  - David Wendt (https://github.com/davidwendt)

URL: rapidsai#20836
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team improvement Improvement / enhancement to an existing function libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants