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

[Java][CI] Use apache/arrow-java for integration test #44831

Closed
kou opened this issue Nov 25, 2024 · 1 comment · Fixed by #44932
Closed

[Java][CI] Use apache/arrow-java for integration test #44831

kou opened this issue Nov 25, 2024 · 1 comment · Fixed by #44932
Assignees
@lidavidm
Labels
Component: Continuous Integration Component: Java Type: enhancement
Milestone
19.0.0

Comments

@kou
Copy link
Member

kou commented Nov 25, 2024

Describe the enhancement requested

After apache/arrow-java works.

We should use apache/arrow-java like apache/arrow-go:

arrow/.github/workflows/integration.yml

Line 117 in a6fe595

-e ARCHERY_INTEGRATION_WITH_GO=1 \

Component(s)

Continuous Integration, Java
@lidavidm lidavidm self-assigned this Dec 4, 2024
lidavidm added a commit to lidavidm/arrow that referenced this issue Dec 4, 2024
@lidavidm
apacheGH-44831: [CI] Build Java for integration tests from arrow-java
4724339
@github-actions github-actions bot mentioned this issue Dec 4, 2024
Merged
lidavidm added a commit to lidavidm/arrow that referenced this issue Dec 4, 2024
@lidavidm
apacheGH-44831: [CI] Build Java for integration tests from arrow-java
9191caf
lidavidm added a commit to lidavidm/arrow that referenced this issue Dec 4, 2024
@lidavidm
apacheGH-44831: [CI] Build Java for integration tests from arrow-java
183a276
lidavidm added a commit to lidavidm/arrow that referenced this issue Dec 4, 2024
@lidavidm
apacheGH-44831: [CI] Build Java for integration tests from arrow-java
58faf95
lidavidm added a commit to lidavidm/arrow that referenced this issue Dec 5, 2024
@lidavidm
apacheGH-44831: [CI] Build Java for integration tests from arrow-java
3b6f2b0
@lidavidm lidavidm linked a pull request Dec 5, 2024 that will close this issue
GH-44831: [CI] Build Java for integration tests from arrow-java #44932
Merged
kou pushed a commit that referenced this issue Dec 5, 2024
@lidavidm
GH-44831: [CI] Build Java for integration tests from arrow-java (#44932)
8ac434f 
### Rationale for this change

We moved `java/` in apache/arrow to apache/arrow-java. We should use apache/arrow-java to integration tests.

### What changes are included in this PR?

* Remove `java/`
* Use apache/arrow-java for integration test

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.

* GitHub Issue: #44945
* GitHub Issue: #44831

Authored-by: David Li <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
@kou kou closed this as completed in #44932 Dec 5, 2024
@kou kou added this to the 19.0.0 milestone Dec 5, 2024
@kou
Copy link
Member Author

kou commented Dec 5, 2024

Issue resolved by pull request 44932
#44932

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lidavidm lidavidm

Labels
Component: Continuous Integration Component: Java Type: enhancement
Projects
None yet
Milestone
19.0.0
Development

Successfully merging a pull request may close this issue.

GH-44831: [CI] Build Java for integration tests from arrow-java lidavidm/arrow
2 participants
@kou @lidavidm