Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16968 )

Change subject: [build] Fix occasional Gradle build failures
......................................................................


Patch Set 2: Code-Review+2

(2 comments)

http://gerrit.cloudera.org:8080/#/c/16968/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16968/1//COMMIT_MSG@10
PS1, Line 10: cryptic often Protobuf
> I have seen a few different exceptions occur depending on the race/timing.
Ack


http://gerrit.cloudera.org:8080/#/c/16968/1/CMakeLists.txt
File CMakeLists.txt:

http://gerrit.cloudera.org:8080/#/c/16968/1/CMakeLists.txt@127
PS1, Line 127: add_custom_command(OUTPUT ${PROTO_JAR}
             :         COMMAND ./gradlew :kudu-proto:jar ${GRADLE_FLAGS}
             :         WORKING_DIRECTORY "${JAVA_DIR}"
             :         DEPENDS init_gradle)
> Yes your description is correct. For some reason I thought both modules use
Thanks for confirming!



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iefa02e52a74f8a60c3029188995574ffdb0d3d5c
Gerrit-Change-Number: 16968
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 25 Jan 2021 21:53:30 +0000
Gerrit-HasComments: Yes

Reply via email to