Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11715 )
Change subject: Follow-up improvements to locate_row ...................................................................... Follow-up improvements to locate_row - Added test coverage for a string literal with a space in it. - Abbreviated the tests by adding a couple of lambdas. - Print out all tablet ids when multiple tablets match the primary key. This should never happen, but if it does this output will be useful. - Couple of other small, non-functional changes. Change-Id: I9f4cca87963eee1d6da45fcbd3c27f782c35e6f6 Reviewed-on: http://gerrit.cloudera.org:8080/11715 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/tools/kudu-admin-test.cc M src/kudu/tools/tool_action_table.cc 2 files changed, 71 insertions(+), 143 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11715 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9f4cca87963eee1d6da45fcbd3c27f782c35e6f6 Gerrit-Change-Number: 11715 Gerrit-PatchSet: 3 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <[email protected]>
