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

Fix Spark 3.1.0 shim build error in GpuHashJoin #546

Merged
merged 1 commit into from
Aug 11, 2020

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Aug 11, 2020

Signed-off-by: Jason Lowe jlowe@nvidia.com

This fixes the recent Spark 3.1.0 shim build failure in GpuHashJoin. Apache Spark added a new case class for the return result from prepareRelation which every HashJoin must implement. Unfortunately that case class is private to the org.apache.spark.sql.execution.joins package which required some refactoring of our code to be able to access that class and thus successfully derive a new join class.

Signed-off-by: Jason Lowe <jlowe@nvidia.com>
@jlowe jlowe added the build Related to CI / CD or cleanly building label Aug 11, 2020
@jlowe jlowe added this to the Aug 3 - Aug 14 milestone Aug 11, 2020
@jlowe jlowe self-assigned this Aug 11, 2020
@jlowe
Copy link
Member Author

jlowe commented Aug 11, 2020

build

@jlowe jlowe merged commit 71d49f6 into NVIDIA:branch-0.2 Aug 11, 2020
@jlowe jlowe deleted the fix-310-joins branch October 16, 2020 20:30
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
…IDIA#546)

Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>

Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants