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 Alias to shims #1272

Merged
merged 3 commits into from
Dec 7, 2020
Merged

Add Alias to shims #1272

merged 3 commits into from
Dec 7, 2020

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Dec 4, 2020

Apache Spark 3.1.0 recently changed the signature of Alias, so it needs to be moved to the shims. I also fixed a missing SortOrder change that was failing in the integration tests when testing against Spark 3.1.

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

jlowe commented Dec 4, 2020

build

@jlowe
Copy link
Member Author

jlowe commented Dec 4, 2020

build

@jlowe
Copy link
Member Author

jlowe commented Dec 5, 2020

build

@sameerz sameerz modified the milestones: Nov 23 - Dec 4, Dec 7 - Dec 18 Dec 7, 2020
@revans2 revans2 merged commit cce1ec1 into NVIDIA:branch-0.3 Dec 7, 2020
@sameerz sameerz added the Spark 3.1+ Bugs only related to Spark 3.1 or higher label Dec 12, 2020
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>
@jlowe jlowe deleted the shim-alias branch September 10, 2021 15:31
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
…IDIA#1272)

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 Spark 3.1+ Bugs only related to Spark 3.1 or higher
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants