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

Add support for ReplicateRows #4388

Merged
merged 6 commits into from
Dec 21, 2021
Merged

Conversation

nartal1
Copy link
Collaborator

@nartal1 nartal1 commented Dec 20, 2021

This fixes #4104 . ReplicateRows is used as optimization for exceptAll and intersectAll. The final plan includes HashAggregateExec followed by replicateRows.
Current limitation of ReplicateRows would be that we could support only those datatypes which are supported by HashAggregateExec. Added integration tests for both exceptAll and intersectAll

Signed-off-by: Niranjan Artal <nartal@nvidia.com>
Signed-off-by: Niranjan Artal <nartal@nvidia.com>
@nartal1 nartal1 added the feature request New feature or request label Dec 20, 2021
@nartal1 nartal1 added this to the Dec 13 - Jan 7 milestone Dec 20, 2021
@nartal1 nartal1 requested a review from revans2 December 20, 2021 06:45
@nartal1 nartal1 self-assigned this Dec 20, 2021
@nartal1
Copy link
Collaborator Author

nartal1 commented Dec 20, 2021

build

Signed-off-by: Niranjan Artal <nartal@nvidia.com>
@nartal1
Copy link
Collaborator Author

nartal1 commented Dec 21, 2021

build

revans2
revans2 previously approved these changes Dec 21, 2021
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.

Just a very minor nit, and a follow on issue that I want to be sure we don't forget to do.

Signed-off-by: Niranjan Artal <nartal@nvidia.com>
@nartal1
Copy link
Collaborator Author

nartal1 commented Dec 21, 2021

@revans2 Filed follow on issue for decimal 128 #4410 . PTAL

@nartal1
Copy link
Collaborator Author

nartal1 commented Dec 21, 2021

build

@nartal1 nartal1 merged commit 812e463 into NVIDIA:branch-22.02 Dec 21, 2021
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.

[FEA] Support org.apache.spark.sql.catalyst.expressions.ReplicateRows
2 participants