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

GH-44843: [CI][Doc] Remove building Java documentation from this repository #45000

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

raulcd
Copy link
Member

@raulcd raulcd commented Dec 11, 2024

Rationale for this change

Our current documentation jobs are failing to build.

What changes are included in this PR?

Do not build Java documentation. This will be build on the arrow-java repository:

Are these changes tested?

Yes on CI

Are there any user-facing changes?

It shouldn't. We won't be producing the documentation here but it should be generated on the arrow-java repository.

Copy link

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

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Dec 11, 2024
@raulcd
Copy link
Member Author

raulcd commented Dec 11, 2024

From my understanding ci/scripts/java_build.sh is still used for building Java for integration tests and for conbench (even though conbench tests are not working for Java since the migration).

@raulcd
Copy link
Member Author

raulcd commented Dec 11, 2024

@github-actions crossbow submit test-debian-12-docs

Copy link

Revision: d6f8587

Submitted crossbow builds: ursacomputing/crossbow @ actions-28cfc978fe

Task Status
test-debian-12-docs GitHub Actions

@kou
Copy link
Member

kou commented Dec 12, 2024

From my understanding ci/scripts/java_build.sh is still used for building Java for integration tests and for conbench (even though conbench tests are not working for Java since the migration).

We should use https://github.com/apache/arrow-java/blob/main/ci/scripts/java_build.sh instead.

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 a075bc0 into apache:main Dec 12, 2024
36 checks passed
@kou kou removed the awaiting committer review Awaiting committer review label Dec 12, 2024
@github-actions github-actions bot added the awaiting merge Awaiting merge label Dec 12, 2024
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