You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2019/11/02 21:15:21 UTC

[kudu-CR] KUDU-2986 p1: adjust the kudu CLI output from 0 to N/A

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14567 )

Change subject: KUDU-2986 p1: adjust the kudu CLI output from 0 to N/A
......................................................................


Patch Set 3: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14567/2/src/kudu/tools/kudu-tool-test.cc
File src/kudu/tools/kudu-tool-test.cc:

http://gerrit.cloudera.org:8080/#/c/14567/2/src/kudu/tools/kudu-tool-test.cc@5574
PS2, Line 5574: TEST_F(ToolTest, TestGetTableStatisticsLiveRowCountNotSupported) {
> Yep, I think below two test cases are enough:
Great, thank you!



-- 
To view, visit http://gerrit.cloudera.org:8080/14567
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I82c47f5ac07cf63d455c0720614c5383bfa88ca3
Gerrit-Change-Number: 14567
Gerrit-PatchSet: 3
Gerrit-Owner: helifu <hz...@corp.netease.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: helifu <hz...@corp.netease.com>
Gerrit-Comment-Date: Sat, 02 Nov 2019 21:15:21 +0000
Gerrit-HasComments: Yes