Jim Apple has posted comments on this change. Change subject: Set version of CDH dependencies to cdh5.11.0-SNAPSHOT. ......................................................................
Patch Set 3: > The ninja part and the "all their valid combinations" was not clear > to me. It's great that we have additional validation, but I doubt > my change is going to break ninja. To me this looks more like a bug > in this Jenkins job, and I don't think it's necessary for me to fix > the GVO before merging this change which unblocks the rather > important integration job. I don't necessarily think the external/public GVO job needs to be fixed first, either. When I asked "Do you think this was spurious?" I was being genuine. It could be a ninja bug, it could be a build script bug, it could be isolated to impala-lzo, it could be a Jenkins bug, it could be a bug with this jenkins job config, and so on. -- To view, visit http://gerrit.cloudera.org:8080/5540 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6b23fb1e4c041c377725a22e313d8bf747205e31 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-Reviewer: Taras Bobrovytsky <tbobrovyt...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-HasComments: No