Re: [PR] [VL][CI] Update gpu image to centos-9-jdk17-cudf [gluten]

2026-08-04 Thread via GitHub


Copilot commented on code in PR #12689:
URL: https://github.com/apache/gluten/pull/12689#discussion_r3711425461


##
dev/docker/cudf/Dockerfile:
##


Review Comment:
   This PR is described as a JDK version/tag update for the GPU image, but this 
change also enables S3/GCS/ABFS in the cuDF image build. That’s a 
functional/feature change that can affect build time, image size, and 
dependencies. Consider splitting this into a separate PR (or update the PR 
title/description to reflect the additional scope and include rationale for 
enabling these features by default in the GPU image).



-- 
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]



[PR] [VL][CI] Update gpu image to centos-9-jdk17-cudf [gluten]

2026-08-04 Thread via GitHub


marin-ma opened a new pull request, #12689:
URL: https://github.com/apache/gluten/pull/12689

   The jdk version in the gpu image is now jdk17 after 
https://github.com/apache/gluten/pull/11835


-- 
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]