Mike Percy has posted comments on this change. Change subject: thirdparty: Clean up thirdparty prefix from previous builds ......................................................................
Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5826/2/thirdparty/build-thirdparty.sh File thirdparty/build-thirdparty.sh: Line 105: local INIT_FLAG="" > I think the flag passing mechanism is unnecessarily generic for the very li There's not much difference between this code and what you suggested, right? Anyway, I simplified it to what you suggested. http://gerrit.cloudera.org:8080/#/c/5826/2/thirdparty/postflight.py File thirdparty/postflight.py: PS2, Line 21: Currently, the only check is a tsan-related check. It is only executed when the : string "tsan" is passed as an argument when executing this script. > Why add this level of specificity? It'll become stale and require an update Meh, I like adding docs to script. But it's not that important. I'll just remove it. -- To view, visit http://gerrit.cloudera.org:8080/5826 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: If2a7ceb3dad1acede152775611f549798723016c Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-HasComments: Yes
