merlimat commented on a change in pull request #7549:
URL: https://github.com/apache/pulsar/pull/7549#discussion_r455451284
##########
File path: .github/workflows/ci-cpp.yaml
##########
@@ -70,9 +70,9 @@ jobs:
docker rmi $(docker images -q) -f
df -h
- - name: build pacakge
+ - name: build package
if: steps.docs.outputs.changed_only == 'no'
- run: mvn -B -Dorg.slf4j.simpleLogger.defaultLogLevel=ERROR package
-DskipTests
+ run: mvn -B -ntp -Dorg.slf4j.simpleLogger.defaultLogLevel=ERROR -ntp
install -Pcore-modules -DskipTests
Review comment:
I think all the jobs that are running broker tests can also use just
core-modules
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
