xvrl commented on code in PR #15106:
URL: https://github.com/apache/druid/pull/15106#discussion_r1370439341
##########
.github/workflows/static-checks.yml:
##########
@@ -51,12 +51,33 @@ jobs:
with:
distribution: 'zulu'
java-version: ${{ matrix.java }}
- cache: 'maven'
+
+ - name: restore maven dependencies
+ # restore maven dependencies only. 'depsonly' keys are safe to restore
+ # since they don't contain any artifacts created by mvn install
+ - uses: actions/cache/restore@v3
+ with:
+ key: maven-${{ runner.os }}-depsonly-${{ hashFiles('**/pom.xml') }}
+ path: ~/.m2/repository
+ restore-keys: |
+ maven-${{ runner.os }}-depsonly-
+
+ - name: compile and cache maven dependencies
+ # run maven compile step to cache all shared maven dependencies
+ # maven install will be run in a subsequent step to avoid polluting
the global maven cache
Review Comment:
excluding artifacts is brittle since it requires knowing every artifact that
the project might write to the local maven repo, and knowing what might or
might not be safe to cache.
Instead I'd rather we err on the safe side and only add things we know are
safe to cache.
--
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]
