Skip to content

Conversation

@lidavidm
Copy link
Member

@lidavidm lidavidm commented Nov 5, 2024

Rationale for this change

I missed that gRPC (Protobuf) cannot be upgraded due to one of our CI jobs using an older libstdc++.

What changes are included in this PR?

This reverts commit 1a6de9d.

Are these changes tested?

Yes

Are there any user-facing changes?

No

@lidavidm
Copy link
Member Author

lidavidm commented Nov 5, 2024

@github-actions crossbow submit -g java

@github-actions
Copy link

github-actions bot commented Nov 5, 2024

Revision: 7426b87

Submitted crossbow builds: ursacomputing/crossbow @ actions-422c28145e

Task Status
java-jars GitHub Actions
test-conda-python-3.11-spark-master GitHub Actions
verify-rc-source-java-linux-almalinux-8-amd64 GitHub Actions
verify-rc-source-java-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-20.04-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-22.04-amd64 GitHub Actions
verify-rc-source-java-macos-amd64 GitHub Actions

@lidavidm lidavidm merged commit f3b8d6b into apache:main Nov 5, 2024
@lidavidm lidavidm removed the awaiting committer review Awaiting committer review label Nov 5, 2024
@lidavidm lidavidm deleted the minor-grpc branch November 5, 2024 23:14
@conbench-apache-arrow
Copy link

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

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
### Rationale for this change

I missed that gRPC (Protobuf) cannot be upgraded due to one of our CI jobs using an older libstdc++.

### What changes are included in this PR?

This reverts commit 1a6de9d.

### Are these changes tested?

Yes

### Are there any user-facing changes?

No

Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants