Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14702 )

Change subject: [mini-cluster] fix on building mini-cluster JAR
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/14702/1/build-support/mini-cluster/build_mini_cluster_binaries.sh
File build-support/mini-cluster/build_mini_cluster_binaries.sh:

http://gerrit.cloudera.org:8080/#/c/14702/1/build-support/mini-cluster/build_mini_cluster_binaries.sh@81
PS1, Line 81:   mv $BUILD_ROOT $BUILD_ROOT.$suffix
> Maybe log this action?
Done


http://gerrit.cloudera.org:8080/#/c/14702/1/build-support/mini-cluster/build_mini_cluster_binaries.sh@134
PS1, Line 134:   echo "ERROR: this script can handle only single kudu-binary 
artifact;"
> Maybe log all of the matching artifacts?
Done



--
To view, visit http://gerrit.cloudera.org:8080/14702
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic2d5c2a83e2a0ab959e2eb7898d47ab736cd1e1f
Gerrit-Change-Number: 14702
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 13 Nov 2019 15:23:14 +0000
Gerrit-HasComments: Yes

Reply via email to