Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11373 )
Change subject: KUDU-2529: Add a "-tables=<tables>" flag to the "kudu table list". ...................................................................... Patch Set 1: Looks like when you published your new commit, you dropped the Change-Id line and your post-commit hook generated a new one, which resulted in a brand new Gerrit change. Could you amend this commit to replace the Change-Id with the one found in https://gerrit.cloudera.org/c/11360? That way we can continue reviewing the patch in the same place, and the interdiff between patch revisions will work. Also, you've got some test failures to fix, as well as some clang-tidy suggestions. -- To view, visit http://gerrit.cloudera.org:8080/11373 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I719902ae8b731e2bc77ab53c087db7fd289a980f Gerrit-Change-Number: 11373 Gerrit-PatchSet: 1 Gerrit-Owner: helifu <hzhel...@corp.netease.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Fri, 31 Aug 2018 17:38:04 +0000 Gerrit-HasComments: No