Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14470 )
Change subject: master: GetTableStatistics should use signed ints ...................................................................... master: GetTableStatistics should use signed ints The statistics are reported to the master as signed ints, so the master should likewise advertise them as such. Otherwise the live_row_count value of -1 (when a tablet doesn't support it) shows up as: TABLE test-workload on disk size: 0 live row count: 18446744073709551615 The public APIs were already treating them as signed; they were only only unsigned on the wire. Change-Id: I398cb85888bdc9463264692911788de2fef67dfc Reviewed-on: http://gerrit.cloudera.org:8080/14463 Reviewed-by: Grant Henke <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit adecebd9e22b50254d69c94c784d5cba3657890b) Reviewed-on: http://gerrit.cloudera.org:8080/14470 --- M src/kudu/client/table_statistics-internal.h M src/kudu/master/master.proto M src/kudu/tools/kudu-tool-test.cc 3 files changed, 24 insertions(+), 7 deletions(-) Approvals: Grant Henke: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14470 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.11.x Gerrit-MessageType: merged Gerrit-Change-Id: I398cb85888bdc9463264692911788de2fef67dfc Gerrit-Change-Number: 14470 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
