chore(ci): Advance velox#27802
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis pull request advances the Velox submodule reference used by presto-native-execution to a newer upstream commit, with no additional code or configuration changes in this repository. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
The failure is valid. Analysis from Co-pilot. The failure is caused by a stale Homebrew package reference in the workflow, not by the C++ build itself. Root causeThe job completes dependency builds and then fails at: brew link --force protobuf@21Error from the log: This means the macOS runner no longer has
brew link --force protobuf@21SolutionUpdate the workflow to stop linking a versioned formula that no longer exists on the runner. Recommended fixReplace: brew link --force protobuf@21with: brew link --overwrite protobufor, if linking is unnecessary, remove the line entirely. Suggested patchIn @@
- brew link --force protobuf@21
+ brew link --overwrite protobufWhy this is the right fix
If Presto specifically requires protobuf 21If that version is truly required, make the workflow install it before linking, but that is less robust because Homebrew may have dropped the versioned formula. For example: brew install protobuf
brew link --overwrite protobufFocused recommendationMake this one-line workflow change: brew link --overwrite protobufat |
Summary by Sourcery
Chores: