Skip to content

GH-48594: [C++][FlightRPC] Fix ODBC CI Caching Issue #48595

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
alinaliBQ wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

GH-48594: [C++][FlightRPC] Fix ODBC CI Caching Issue #48595

alinaliBQ wants to merge 10 commits into from
+180 −10

Conversation

@alinaliBQ
Copy link
Contributor

@alinaliBQ alinaliBQ commented Dec 18, 2025 •
edited by github-actions bot
Loading

Rationale for this change

#48594

What changes are included in this PR?

  • Resolve caching issue inside ODBC CI

Are these changes tested?

  • Tested in CI

Are there any user-facing changes?

N/A

@github-actions
Copy link

github-actions bot commented Dec 18, 2025

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

@github-actions github-actions bot added the awaiting review Awaiting review label Dec 18, 2025
@alinaliBQ
Attempt to resolve caching issue with ODBC CI
64db26d 
Add write permission to store vcpkg cache
@alinaliBQ alinaliBQ force-pushed the gh-48594-fix-msvc-ci branch 6 times, most recently from 33227e6 to 6efa7d4 Compare December 18, 2025 23:07
@alinaliBQ
Check nuget paths in Ruby Glib and ODBC
cb281bf 
Run vcpkg install after vcpkg is set in GLib
@alinaliBQ
Enable vcpkg verbose in GLib
c32f221 
* confirmed `vcpkg install` command returns 403 error
@alinaliBQ alinaliBQ force-pushed the gh-48594-fix-msvc-ci branch from 6efa7d4 to c32f221 Compare December 18, 2025 23:32
@alinaliBQ
Add ARROW_HOME
cb6a294 
Remove `ARROW_SIMD_LEVEL` as it is not required for ODBC.
@alinaliBQ
Remove debug msgs
d0bd9f6 
Cannot get vcpkg in cpp_build to show debug messages
@alinaliBQ alinaliBQ force-pushed the gh-48594-fix-msvc-ci branch from 4b78fb7 to d0bd9f6 Compare December 19, 2025 00:06
@alinaliBQ
Copy link
Contributor Author

alinaliBQ commented Dec 19, 2025

Hi @pitrou and @raulcd, to follow up on #48313 I have opened this draft PR.

As Raul has pointed out in #48313 (comment), there is an issue pushing the packages to nuget. Therefore, when a new ODBC build begins, all vcpkg packages are rebuilt and that is the root cause of the long ODBC build time.

Does it require any changes in the GitHub repository settings to allow the new workflow to push packages to https://nuget.pkg.github.com/apache?

@alinaliBQ
Copy link
Contributor Author

alinaliBQ commented Dec 19, 2025 •
edited
Loading

During investigation, I tried running vcpkg install command before the cpp_build.sh script in C GLib & Ruby MSVC workflow, and it has the same 403 error as ODBC MSVC. But the script cpp_build.sh in GLib workflow can still publish nuget packages normally. I am not sure why the same script works in GLib MSVC, but not in ODBC MSVC.
Run log: https://github.com/apache/arrow/actions/runs/20353354024/job/58483242851?pr=48595#step:13:3353

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

Reviewers

@assignUser assignUser Awaiting requested review from assignUser assignUser will be requested when the pull request is marked ready for review assignUser is a code owner

@jonkeane jonkeane Awaiting requested review from jonkeane jonkeane will be requested when the pull request is marked ready for review jonkeane is a code owner

@kou kou Awaiting requested review from kou kou will be requested when the pull request is marked ready for review kou is a code owner

@raulcd raulcd Awaiting requested review from raulcd raulcd will be requested when the pull request is marked ready for review raulcd is a code owner

Assignees

No one assigned

Labels

awaiting review Awaiting review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@alinaliBQ