Skip to content

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented May 2, 2025

Rationale for this change

The current dependencies of ArrowFlight are outdated about two years. We had better update them to bring the latest features and bug fixes.

What changes are included in this PR?

  • Upgrade grpc-swift from 1.15.0 to 1.25.0.
  • Upgrade swift-protobuf from 1.25.0 to 1.29.0.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

@dongjoon-hyun dongjoon-hyun requested a review from kou as a code owner May 2, 2025 07:39
@github-actions
Copy link

github-actions bot commented May 2, 2025

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

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit 6006f11 into apache:main May 2, 2025
13 checks passed
@kou kou removed the awaiting review Awaiting review label May 2, 2025
@github-actions github-actions bot added the awaiting merge Awaiting merge label May 2, 2025
@dongjoon-hyun
Copy link
Member Author

Thank you, @kou .

@dongjoon-hyun dongjoon-hyun deleted the GH-46290 branch May 2, 2025 13:11
@conbench-apache-arrow
Copy link

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

There were 3 benchmark results indicating a performance regression:

The full Conbench report has more details.

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