Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16181 )
Change subject: [docker] Use ccache and Gradle cache across builds ...................................................................... [docker] Use ccache and Gradle cache across builds This patch uses buildkits `--mount=type=cache` syntax to allow the builds ccache and Gradle cache to be used across Docker builds. This speeds up the Docker build when re-building the thirdparty, C++, and Java modules and mirrors the best practices for non-Docker based builds. See the buildkit documentation for details: https://github.com/moby/buildkit/blob/master/frontend/dockerfile/docs/experimental.md#run---mounttypecache Below shows the runtime difference for subsequent builds of the thirdparty target and Kudu target before and after this change: `thirdparty` target: - Before: 1:18:13 - After: 0:17:47 `kudu` target: - Before: 0:19:10 - After: 0:04:08 Change-Id: Ie21ed47983b990e9d2aac419454b9a37a0600334 Reviewed-on: http://gerrit.cloudera.org:8080/16181 Tested-by: Kudu Jenkins Reviewed-by: Bankim Bhavsar <[email protected]> Reviewed-by: Greg Solovyev <[email protected]> Reviewed-by: Attila Bukor <[email protected]> --- M docker/Dockerfile M docker/README.adoc M docker/docker-build.py 3 files changed, 40 insertions(+), 13 deletions(-) Approvals: Kudu Jenkins: Verified Bankim Bhavsar: Looks good to me, but someone else must approve Greg Solovyev: Looks good to me, but someone else must approve Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16181 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie21ed47983b990e9d2aac419454b9a37a0600334 Gerrit-Change-Number: 16181 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Greg Solovyev <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
