Skip to content

Conversation

@kou
Copy link
Member

@kou kou commented Jan 8, 2025

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-actions github-actions bot added the awaiting committer review Awaiting committer review label Jan 8, 2025
@kou
Copy link
Member Author

kou commented Jan 8, 2025

+1

@kou kou merged commit 4f4eb3f into apache:main Jan 8, 2025
13 checks passed
@kou kou deleted the ci-integration-java branch January 8, 2025 05:11
@kou kou removed the awaiting committer review Awaiting committer review label Jan 8, 2025
@kou kou modified the milestone: 19.0.0 Jan 8, 2025
@kou
Copy link
Member Author

kou commented Jan 8, 2025

Oh, I associated to wrong issue...
It should be #45198 not #45164.
I'll rename the PR title but merged commit's message can't be changed...

@amoeba We need this for 19.0.0 to avoid integration build failure. Sorry.

@kou kou changed the title GH-45164: [CI][Integration] Follow build script name change in apache/arrow-java GH-45198: [CI][Integration] Follow build script name change in apache/arrow-java Jan 8, 2025
@github-actions
Copy link

github-actions bot commented Jan 8, 2025

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

amoeba pushed a commit that referenced this pull request Jan 9, 2025
…/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 <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
amoeba pushed a commit that referenced this pull request Jan 11, 2025
…/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 <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
@conbench-apache-arrow
Copy link

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

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 3 possible false positives for unstable benchmarks that are known to sometimes produce them.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant