Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22144 )
Change subject: [build-and-test] run 'gradle clean' before building Java ...................................................................... [build-and-test] run 'gradle clean' before building Java When updating Java dependencies in changelists [1] and [2], I noticed that pre-commit builds were failing fail because the copyDistTestJars task wouldn't copy all the necessary JARs to into the staging directory for dist-test since gradle would assume the target is up-to-date. I think that was due to the fact that the same workspace was re-used between different builds, and no 'clean' task was run. This changelist is to address the issue. To avoid incurring the penalty of rebuilding Java bits from scratch, this patch enables gradle's build cache. As per [3], this is exactly the use-case to benefit from gradle's caching. In addition, this patch re-enables parallel builds as well. [1] https://github.com/apache/kudu/commit/b009144cd [2] https://github.com/apache/kudu/commit/150ec7ff5 [3] https://docs.gradle.org/current/userguide/part6_gradle_caching.html Change-Id: Ief3d2f07dfcf8746aa69f870c319599c92e098dc Reviewed-on: http://gerrit.cloudera.org:8080/22144 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> --- M build-support/jenkins/build-and-test.sh 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Alexey Serbin: Verified Abhishek Chennaka: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/22144 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ief3d2f07dfcf8746aa69f870c319599c92e098dc Gerrit-Change-Number: 22144 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <[email protected]>
