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.

Sign up for GitHub

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

Jump to bottom

Consolidate deps switching in an intermediate pom [databricks] #9508

Merged
merged 12 commits into from
Oct 31, 2023
Merged

Consolidate deps switching in an intermediate pom [databricks] #9508

merged 12 commits into from
Oct 31, 2023

Conversation

gerashegalov
Copy link
Collaborator

@gerashegalov gerashegalov commented Oct 22, 2023 •
edited
Loading

Fixes #9552

  • Factor out dependency switching profiles for different Spark builds into a single intermediate parent pom

Signed-off-by: @gerashegalov

@gerashegalov gerashegalov added build Related to CI / CD or cleanly building tech debt labels Oct 22, 2023
@gerashegalov gerashegalov self-assigned this Oct 22, 2023
@gerashegalov
Copy link
Collaborator Author

build

@gerashegalov gerashegalov added the bug Something isn't working label Oct 26, 2023
@gerashegalov gerashegalov mentioned this pull request Oct 26, 2023
Closed
@gerashegalov gerashegalov changed the title [WIP] Consolidate deps switching in an intermediate pom [databricks] Consolidate deps switching in an intermediate pom [databricks] Oct 27, 2023
@gerashegalov
regenerate 2.13 poms
7ae26ba 
Signed-off-by: Gera Shegalov <[email protected]>
@gerashegalov gerashegalov marked this pull request as ready for review October 27, 2023 19:37
@gerashegalov
Copy link
Collaborator Author

build

@gerashegalov gerashegalov mentioned this pull request Oct 28, 2023
Closed
@gerashegalov
Copy link
Collaborator Author

build

1 similar comment
@gerashegalov
Copy link
Collaborator Author

build

@gerashegalov gerashegalov merged commit 51f933f into NVIDIA:branch-23.12 Oct 31, 2023
36 of 37 checks passed
@gerashegalov gerashegalov mentioned this pull request Nov 12, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jlowe jlowe jlowe approved these changes

@revans2 revans2 Awaiting requested review from revans2 revans2 is a code owner

@tgravescs tgravescs Awaiting requested review from tgravescs tgravescs is a code owner

@GaryShen2008 GaryShen2008 Awaiting requested review from GaryShen2008 GaryShen2008 is a code owner

@NvTimLiu NvTimLiu Awaiting requested review from NvTimLiu NvTimLiu is a code owner

@pxLi pxLi Awaiting requested review from pxLi pxLi is a code owner

Assignees

@gerashegalov gerashegalov

Labels
bug Something isn't working build Related to CI / CD or cleanly building tech debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] Inconsistent CDH dependency overrides across submodules
2 participants
@gerashegalov @jlowe