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.

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

Support for LeftOuter/BuildRight and RightOuter/BuildLeft nested loop joins #3242

Merged
merged 4 commits into from
Aug 20, 2021

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Aug 17, 2021

Part of #3111.

This adds GPU support for LeftOuter with BuildRight and RightOuter with BuildLeft broadcast nested loop joins.

… joins

Signed-off-by: Jason Lowe <jlowe@nvidia.com>
@jlowe jlowe added the feature request New feature or request label Aug 17, 2021
@jlowe jlowe added this to the Aug 16 - Aug 27 milestone Aug 17, 2021
@jlowe jlowe self-assigned this Aug 17, 2021
@jlowe
Copy link
Member Author

jlowe commented Aug 17, 2021

build

Copy link
Collaborator

@revans2 revans2 left a comment

Choose a reason for hiding this comment

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

It looks good, just curious about the TODOs in the code

@jlowe
Copy link
Member Author

jlowe commented Aug 18, 2021

build

revans2
revans2 previously approved these changes Aug 19, 2021
@jlowe
Copy link
Member Author

jlowe commented Aug 19, 2021

Had to resolve a conflict with yesterday's AST API update.

@jlowe
Copy link
Member Author

jlowe commented Aug 19, 2021

build

@sameerz
Copy link
Collaborator

sameerz commented Aug 19, 2021

build

@revans2 revans2 merged commit 92a200e into NVIDIA:branch-21.10 Aug 20, 2021
razajafri pushed a commit to razajafri/spark-rapids that referenced this pull request Aug 23, 2021
… joins (NVIDIA#3242)

Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@jlowe jlowe deleted the nested-loop-join-left-right branch September 10, 2021 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants