Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Unshim many SparkShim interfaces [databricks] #5031

Merged
merged 17 commits into from
Mar 24, 2022

Conversation

jlowe
Copy link
Contributor

@jlowe jlowe commented Mar 23, 2022

This is part of #5005. This removes the remaining SparkShim interfaces that are now common across supported Spark versions. There are still exec and expr overrides that can be unshimmed which will be addressed in a followup.

jlowe added 15 commits March 23, 2022 16:01
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
@jlowe jlowe added this to the Mar 21 - Apr 1 milestone Mar 23, 2022
@jlowe jlowe self-assigned this Mar 23, 2022
@jlowe
Copy link
Contributor Author

jlowe commented Mar 23, 2022

build

@jlowe jlowe changed the title Unshim many SparkShim interfaces Unshim many SparkShim interfaces [databricks] Mar 23, 2022
@jlowe
Copy link
Contributor Author

jlowe commented Mar 23, 2022

build

@jlowe
Copy link
Contributor Author

jlowe commented Mar 23, 2022

build

Copy link
Collaborator

@gerashegalov gerashegalov left a comment

Choose a reason for hiding this comment

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

LGTM

@jlowe jlowe merged commit 0d7750d into NVIDIA:branch-22.06 Mar 24, 2022
@jlowe jlowe deleted the unshim-31x-part2 branch March 24, 2022 14:49
@sameerz sameerz added the task Work required that improves the product but is not user facing label Mar 26, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
task Work required that improves the product but is not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants