Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11576 )
Change subject: Build cleanup in preperation for release ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/11576/3/RELEASING.adoc File RELEASING.adoc: http://gerrit.cloudera.org:8080/#/c/11576/3/RELEASING.adoc@161 PS3, Line 161: gradle clean uploadArchives -PmavenUsername="<APACHE-LDAP-USERNAME>" -PmavenPassword="<APACHE-LDAP-PASSWORD>" > I know the original instructions also used the clean target, but is it real The assemble task doesn't run tests so no form of skip tests is needed. I think the clean is just to ensure there is no chance for any bad/corrupt state. Given this is generating a permanent release, the couple of minutes of extra time is likely worth the safety. -- To view, visit http://gerrit.cloudera.org:8080/11576 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I50c3b9358896b264fd525de558b6be7ef77145e2 Gerrit-Change-Number: 11576 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Comment-Date: Wed, 03 Oct 2018 20:03:41 +0000 Gerrit-HasComments: Yes