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 auto merge conflict 10523 #10528

Merged

Conversation

NvTimLiu
Copy link
Collaborator

@NvTimLiu NvTimLiu commented Mar 1, 2024

Resolve auto-merge conflict at #10523.

Keep version 24.04.0-SNAPSHOT in pom.xml

Keep branch-24.04 351 noSnapshot shim in pom.xml

NOTE: this must be merged as create a merge commit

NvTimLiu and others added 4 commits March 1, 2024 07:30
* Add missed shims for scala2.13

Signed-off-by: Tim Liu <timl@nvidia.com>

* Add 351 snapshot shim for the scala2.13 version of plugin jar

Signed-off-by: Tim Liu <timl@nvidia.com>

* Remove 351 snapshot shim as spark 3.5.1 has been released

Signed-off-by: Tim Liu <timl@nvidia.com>

* Remove scala2.13 351 snapshot shim

Signed-off-by: Tim Liu <timl@nvidia.com>

* Remove 351 shim's jason string

Ran `mvn generate-sources -Dshimplify=true -Dshimplify.move=true -Dshimplify.remove.shim=351`

to remove 351 shim's jason string, and fix some unnecessary empty lines that were introduced

Signed-off-by: Tim Liu <timl@nvidia.com>

* Update Copyright 2024

Auto copyright by below scripts
```
export SPARK_RAPIDS_AUTO_COPYRIGHTER=ON

./scripts/auto-copyrighter.sh $(git diff --name-only origin/branch-24.04..HEAD)
```

Signed-off-by: Tim Liu <timl@nvidia.com>

* Revert "Update Copyright 2024"

This reverts commit 8482847.

* Revert "Remove 351 shim's jason string"

This reverts commit 78d1f00.

* skip 351 from strict checking

* Alien scala2.13/pom.xml to scala2.12 one

Run the script `bash build/make-scala-version-build-files.sh 2.13`

Signed-off-by: Tim Liu <timl@nvidia.com>

* pretend 351 is a snapshot in 24.02

Signed-off-by: Gera Shegalov <gera@apache.org>

* pretend 351 is a SNAPSHOT version

* Revert change of build/shimplify.py

Signed-off-by: Tim Liu <timl@nvidia.com>

---------

Signed-off-by: Tim Liu <timl@nvidia.com>
Signed-off-by: Gera Shegalov <gera@apache.org>
Co-authored-by: Raza Jafri <rjafri@nvidia.com>
Co-authored-by: Gera Shegalov <gera@apache.org>
Signed-off-by: Tim Liu <timl@nvidia.com>
@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Mar 1, 2024

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

@NvTimLiu NvTimLiu merged commit 358b105 into NVIDIA:branch-24.04 Mar 2, 2024
42 of 44 checks passed
@NvTimLiu NvTimLiu deleted the fix-auto-merge-conflict-10523 branch March 2, 2024 02:09
@sameerz sameerz added the task Work required that improves the product but is not user facing label Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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.

None yet

4 participants