Copilot commented on code in PR #13085:
URL: https://github.com/apache/gluten/pull/13085#discussion_r4066088898
##########
.github/workflows/velox_nightly.yml:
##########
@@ -48,17 +49,11 @@ jobs:
ccache-almalinux8-release-default
- name: Build Gluten velox third party
run: |
- docker run -v $GITHUB_WORKSPACE:/work -w /work
apache/gluten:vcpkg-almalinux-8-gcc13 bash -c "
- yum install bison -y
- df -a
- cd /work
- export CCACHE_DIR=/work/.ccache
- export CCACHE_MAXSIZE=1G
- bash dev/ci-velox-buildstatic-centos-8.sh
- ccache -s
- mkdir -p /work/.m2/repository/org/apache/arrow/
- cp -r /root/.m2/repository/org/apache/arrow/*
/work/.m2/repository/org/apache/arrow/
- "
+ yum install bison -y
+ df -a
+ export CCACHE_MAXSIZE=1G
+ bash dev/ci-velox-buildstatic-centos-8.sh
+ ccache -s
Review Comment:
The x86 native job still uploads
`velox-arrow-jar-centos-8-x86-${{github.sha}}` from
`.m2/repository/org/apache/arrow/` (lines 63-67), and the three downstream
package jobs download that artifact. Removing the copy from `/root/.m2` leaves
the workspace path empty, so those downloads fail or produce packages without
Arrow; retain the copy as the arm64 job does.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
