-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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.
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
Bump up presto java version in docker container #10254
Conversation
✅ Deploy Preview for meta-velox canceled.
|
Can you also edit velox/.github/workflows/docker.yml Line 18 in f9f6d82
|
6376787
to
f5fa69b
Compare
f5fa69b
to
25b5a07
Compare
cc: @assignUser Do you know why are the docker images still using old presto version ,even though the version is specifically changed here. |
25b5a07
to
f211321
Compare
It seems there is an issue with the file name? https://github.com/facebookincubator/velox/actions/runs/9605868436/job/26498382674?pr=10254#step:5:186 |
@assignUser This is completely expected, the issue was previously why we werent getting it :) |
f211321
to
821fa27
Compare
@kewang1024 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@kewang1024 merged this pull request in e23df19. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
No description provided.