Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/17966
Change subject: Revert "[tool] Add a '-list_statistics' flag to the 'kudu table
list'."
......................................................................
Revert "[tool] Add a '-list_statistics' flag to the 'kudu table list'."
This patch breaks ABI compatibility for Kudu C++ client by adding an
overloaded for the ListTables() method as per [1]:
You cannot...
* add an overload (binary compatible, but not source compatible:
it makes &func ambiguous). Adding overloads to already overloaded
functions is ok (since any use of &func already needed a cast).
[1] https://community.kde.org/Policies/Binary_Compatibility_Issues_With_C%2B%2B
This reverts commit 3ccc434ea6dcbef15a12bd1734c8e66b69a611c5.
Change-Id: Iaec8c6cbebbd931260d7f2d5fa34845701ee490f
---
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/master/catalog_manager.cc
M src/kudu/master/master.proto
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_table.cc
6 files changed, 5 insertions(+), 101 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/66/17966/1
--
To view, visit http://gerrit.cloudera.org:8080/17966
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaec8c6cbebbd931260d7f2d5fa34845701ee490f
Gerrit-Change-Number: 17966
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
