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

[CI][Integration] Follow build script name change in apache/arrow-java #45164

Closed
kou opened this issue Jan 4, 2025 · 1 comment
Closed

[CI][Integration] Follow build script name change in apache/arrow-java #45164

kou opened this issue Jan 4, 2025 · 1 comment
Assignees
@kou
Labels
Component: Continuous Integration Component: Integration Type: enhancement
Milestone
19.0.0

Comments

@kou
Copy link
Member

kou commented Jan 4, 2025

Describe the enhancement requested

https://github.com/apache/arrow/actions/runs/12600313510/job/35119065241#step:11:2724

  + /arrow/java/ci/scripts/java_jni_build.sh /arrow/java /opt/conda/envs/arrow /build/java/ /tmp/dist/java
  /arrow/ci/scripts/integration_arrow_build.sh: line 62: /arrow/java/ci/scripts/java_jni_build.sh: No such file or directory

apache/arrow-java removes java_ prefix from scripts by apache/arrow-java#449 .

Our scripts should follow the change.

Component(s)

Continuous Integration, Integration
kou added a commit to kou/arrow that referenced this issue Jan 4, 2025
@kou
apacheGH-45164: [CI][Integration] Remove "java_" prefix from Java bui…
77fe764 
…ld scripts
@github-actions github-actions bot mentioned this issue Jan 4, 2025
Merged
kou added a commit to kou/arrow that referenced this issue Jan 4, 2025
@kou
apacheGH-45164: [CI][Integration] Remove "java_" prefix from Java bui…
1c508df 
…ld scripts
kou added a commit that referenced this issue Jan 5, 2025
@kou
GH-45164: [CI][Integration] Remove "java_" prefix from Java build scr…
19f0652 
…ipts (#45165)

### Rationale for this change

apache/arrow-java removed `java_` prefix from scripts by apache/arrow-java#449 .

### What changes are included in this PR?

Follow the script name change.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #45164

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 19.0.0 milestone Jan 5, 2025
@kou
Copy link
Member Author

kou commented Jan 5, 2025

Issue resolved by pull request 45165
#45165

@kou kou closed this as completed Jan 5, 2025
@kou kou mentioned this issue Jan 8, 2025
Closed
kou added a commit to kou/arrow that referenced this issue Jan 8, 2025
@kou
apacheGH-45164: [CI][Integration] Follow build script name change in …
428cace 
…apache/arrow-java

For apache/arrow-java#493. Remove `java_` prefix.
@github-actions github-actions bot mentioned this issue Jan 8, 2025
Merged
kou added a commit that referenced this issue Jan 8, 2025
@kou
GH-45164: [CI][Integration] Follow build script name change in apache…
4f4eb3f 
…/arrow-java (#45199)

### Rationale for this change

For apache/arrow-java#493.

### What changes are included in this PR?

Remove `java_` prefix.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #45164

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
amoeba pushed a commit that referenced this issue Jan 9, 2025
@kou @amoeba
GH-45164: [CI][Integration] Follow build script name change in apache…
ffea469 
…/arrow-java (#45199)

### Rationale for this change

For apache/arrow-java#493.

### What changes are included in this PR?

Remove `java_` prefix.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #45164

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
amoeba pushed a commit that referenced this issue Jan 11, 2025
@kou @amoeba
GH-45164: [CI][Integration] Follow build script name change in apache…
18d3341 
…/arrow-java (#45199)

### Rationale for this change

For apache/arrow-java#493.

### What changes are included in this PR?

Remove `java_` prefix.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #45164

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kou kou

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

No branches or pull requests
1 participant
@kou