Huaisi Xu has posted comments on this change. Change subject: IMPALA-3711: Remove unnecessary privilege checks in getDbsMetadata() ......................................................................
Patch Set 21: > It looks like GVM was run against the commit hash from patch set > 20, which is why you got the +1 on it, instead of patch set 21 > along with a submit. > > http://sandbox.jenkins.cloudera.com/job/impala-external-gerrit-verify-merge/2575/ I cannot access sandbox.jenkins.cloudera.com/* now. but according to my shell history, I submitted two gvms. both of them were against 'refs/changes/71/3371/21'. Did I miss anything? GVM messages: huaisi@huaisi-Desktop:~/Impala$ gerrit-verify-merge Running: ssh -p 29418 [email protected] gerrit query --format JSON --current-patch-set c1a1762d00c14fa9d623a1d85918fbd0e1dc3c77 Parsing gerrit refspec from commit: c1a1762d00c14fa9d623a1d85918fbd0e1dc3c77 Launching Jenkins Job using GERRIT_REF_SPEC: 'refs/changes/71/3371/21' TARGET_BRANCH: 'cdh5-trunk' Job submitted successfully. E-mail will be sent to: [email protected] upon completion Not sure why this failed to get merged by gvm. -- To view, visit http://gerrit.cloudera.org:8080/3371 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I17d8c5b9fb12483e4b01b819fba48b6849311a14 Gerrit-PatchSet: 21 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Huaisi Xu <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Huaisi Xu <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-HasComments: No
