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

@raulcdraulcd 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

Copy link
Copy Markdown

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

@raulcdraulcd 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 kouMay 6, 2026 18:07
@github-actionsgithub-actionsBot removed CI: Extra: C++ Run extra C++ CI CI: Extra: Package: Linux Run extra Linux Packages CI labels May 11, 2026
@raulcd

Copy link
Copy Markdown
MemberAuthor

I'll merge this once CI finishes if successful.

@raulcd
raulcd merged commit 9553098 into apache:mainMay 11, 2026
131 of 135 checks passed
@raulcdraulcd removed the awaiting committer review Awaiting committer review label May 11, 2026
@raulcdraulcd mentioned this pull request 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.

@koukou 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-actionsgithub-actionsBot 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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changesAwaiting changesCI: Extra: RRun extra R CIComponent: C++

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@raulcd@kou