Skip to content

Conversation

@hiroyuki-sato
Copy link
Collaborator

@hiroyuki-sato hiroyuki-sato commented May 19, 2025

Rationale for this change

Apache Arrow Java have been separated repository.
CI scripts for Java are unnecessary.

What changes are included in this PR?

Remove CI scripts for Java (Except JNI)

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

@github-actions
Copy link

⚠️ GitHub issue #46500 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions bot added the awaiting review Awaiting review label May 19, 2025
@hiroyuki-sato
Copy link
Collaborator Author

@kou Rest of files are still necessary. Correct? Can we also remove java_build.sh too?

java_build.sh
java_cdata_integration.sh
java_jni_build.sh
java_jni_macos_build.sh
java_jni_manylinux_build.sh
java_jni_windows_build.sh
java_test.sh

@kou
Copy link
Member

kou commented May 19, 2025

We can remove all ci/scripts/java_*.sh.

@hiroyuki-sato hiroyuki-sato changed the title GH-46500: [CI][Java] Remove CI scripts for Java (Except JNI) GH-46500: [CI][Java] Remove CI scripts for Java May 20, 2025
@hiroyuki-sato
Copy link
Collaborator Author

These files also removed.

 delete mode 100755 ci/scripts/java_build.sh
 delete mode 100755 ci/scripts/java_cdata_integration.sh
 delete mode 100755 ci/scripts/java_jni_build.sh
 delete mode 100755 ci/scripts/java_jni_macos_build.sh
 delete mode 100755 ci/scripts/java_jni_manylinux_build.sh
 delete mode 100755 ci/scripts/java_jni_windows_build.sh
 delete mode 100755 ci/scripts/java_test.sh

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

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

+1

@kou kou merged commit a1707db into apache:main May 20, 2025
13 of 15 checks passed
@kou kou removed the awaiting review Awaiting review label May 20, 2025
@github-actions github-actions bot added the awaiting merge Awaiting merge label May 20, 2025
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit a1707db.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

@hiroyuki-sato hiroyuki-sato deleted the topic/remove-java-build branch June 14, 2025 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants