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

Fix merge conflict from branch-24.02 #10544

Merged
merged 5 commits into from
Mar 5, 2024
Merged

Conversation

jlowe
Copy link
Contributor

@jlowe jlowe commented Mar 4, 2024

Fixes the merge conflict flagged in #10543. Only conflicts were pom.xml files where we take the 24.04 version.

NOTE: This needs to be merged with a merge commit, not squashed!

NvTimLiu and others added 5 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>
Signed-off-by: Jason Lowe <jlowe@nvidia.com>
@jlowe jlowe added the build Related to CI / CD or cleanly building label Mar 4, 2024
@jlowe jlowe self-assigned this Mar 4, 2024
@jlowe
Copy link
Contributor Author

jlowe commented Mar 4, 2024

build

@jlowe jlowe merged commit fb7abc8 into NVIDIA:branch-24.04 Mar 5, 2024
43 of 44 checks passed
@jlowe jlowe deleted the fix-merge branch March 5, 2024 14:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants