Skip to content

GH-49938: [C++] Bump bundled c-ares to 1.34.6#49939

Merged
raulcd merged 1 commit into
apache:mainfrom
raulcd:GH-49938
May 11, 2026
Merged

GH-49938: [C++] Bump bundled c-ares to 1.34.6#49939
raulcd merged 1 commit into
apache:mainfrom
raulcd:GH-49938

Conversation

@raulcd

@raulcd raulcd commented May 6, 2026

Copy link
Copy Markdown
Member

Rationale for this change

We are currently using 1.17.2 which is 5 years old. There's been plenty of new versions released we should regularly upgrade our bundled dependencies.

What changes are included in this PR?

Update bundled c-ares from 1.17.2 to 1.34.6

Are these changes tested?

Yes via CI

Are there any user-facing changes?

No

@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown

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

@raulcd raulcd added CI: Extra: C++ Run extra C++ CI CI: Extra: Package: Linux Run extra Linux Packages CI CI: Extra: R Run extra R CI labels May 6, 2026
@raulcd
raulcd marked this pull request as ready for review May 6, 2026 18:06
@raulcd
raulcd requested a review from kou May 6, 2026 18:07
@github-actions github-actions Bot removed CI: Extra: C++ Run extra C++ CI CI: Extra: Package: Linux Run extra Linux Packages CI labels May 11, 2026
@raulcd

raulcd commented May 11, 2026

Copy link
Copy Markdown
Member Author

I'll merge this once CI finishes if successful.

@raulcd
raulcd merged commit 9553098 into apache:main May 11, 2026
131 of 135 checks passed
@raulcd raulcd removed the awaiting committer review Awaiting committer review label May 11, 2026
@raulcd
raulcd deleted the GH-49938 branch May 11, 2026 14:28
@conbench-apache-arrow

Copy link
Copy Markdown

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

None of the specified runs were found on the Conbench server.

The full Conbench report has more details.

@kou kou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

Sorry for missing this!

@github-actions github-actions Bot added the awaiting changes Awaiting changes label May 11, 2026
@conbench-apache-arrow

Copy link
Copy Markdown

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

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.

Mottl pushed a commit to Mottl/arrow that referenced this pull request May 26, 2026
### Rationale for this change

We are currently using 1.17.2 which is 5 years old. There's been plenty of new versions released we should regularly upgrade our bundled dependencies.

### What changes are included in this PR?

Update bundled c-ares from 1.17.2 to 1.34.6

### Are these changes tested?

Yes via CI

### Are there any user-facing changes?

No

* GitHub Issue: apache#49938

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants