zhengruifeng commented on code in PR #48690:
URL: https://github.com/apache/spark/pull/48690#discussion_r1821791771


##########
.github/workflows/build_infra_images_cache.yml:
##########
@@ -27,6 +27,7 @@ on:
     - 'branch-*'
     paths:
     - 'dev/infra/Dockerfile'
+    - 'dev/spark-test-image/docs/Dockerfile'

Review Comment:
   Not yet, because I am not sure how to test again `branch-3.5` in my branches.
   
   After revert, I made such changes in this PR:
   1, keep the location of previous image `dev/infra/Dockerfile`, it will be 
used for both 3.5, and master (except docs);
   2, add file existence check before each step needs the new docs image;
   3, additional url like for docs jobs, it links to previous image for 
`branch-3.5`, and the new image for `master`.
   
   



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

Reply via email to