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] Remove java/ #44945

Closed
kou opened this issue Dec 5, 2024 · 4 comments · Fixed by #44932
Closed

[Java] Remove java/ #44945

kou opened this issue Dec 5, 2024 · 4 comments · Fixed by #44932
Assignees
@lidavidm
Labels
Component: Java Type: enhancement

Comments

@kou
Copy link
Member

kou commented Dec 5, 2024

Describe the enhancement requested

We're moving java/ to https://github.com/apache/arrow-java .

Component(s)

Java
@lidavidm
Copy link
Member

lidavidm commented Dec 5, 2024

I am doing this in #44932. I can try to split it out.

@kou
Copy link
Member Author

kou commented Dec 5, 2024

I think that we need GH-44829 before this. I'll do it.

@kou
Copy link
Member Author

kou commented Dec 5, 2024 •
edited
Loading

Done. Could you work on this?

@lidavidm lidavidm self-assigned this Dec 5, 2024
lidavidm added a commit to lidavidm/arrow that referenced this issue Dec 5, 2024
@lidavidm
apacheGH-44945: [Java] Remove java/
b391570
lidavidm added a commit to lidavidm/arrow that referenced this issue Dec 5, 2024
@lidavidm
apacheGH-44945: [Java] Remove java/
3a46cc2
@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 kou mentioned this issue Dec 5, 2024
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
@raulcd
Copy link
Member

raulcd commented Dec 5, 2024

🥳

@ParthChonkar ParthChonkar mentioned this issue Dec 31, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lidavidm lidavidm

Labels
Component: Java Type: enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

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