imply-elliott commented on code in PR #13962:
URL: https://github.com/apache/druid/pull/13962#discussion_r1153471889


##########
.github/workflows/cron-job-its.yml:
##########
@@ -82,8 +115,16 @@ jobs:
       - name: Checkout branch
         uses: actions/checkout@v3
 
+      - name: setup java

Review Comment:
   That makes sense. However, we can do the same by using the same caching 
parameters the setup java action uses, which just follows this convention: 
https://github.com/actions/cache/blob/0638051e9af2c23d10bb70fa9beffcad6cff9ce3/examples.md#java---maven
   Again, not a big deal, but will save a bit of time downloading and setting 
up java.



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