Hello Tidy Bot, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14142
to look at the new patch set (#4).
Change subject: [test] move filter tests from ksck-test to ksck_remote-test
......................................................................
[test] move filter tests from ksck-test to ksck_remote-test
Some filter tests in ksck-test just tested filter functions
in MockKsckCluster class, but RemoteKsckCluster class which
is used by CLI tool hasn't been tested. It seems unreasonable.
This patch implemented same functional tests in ksck_remote-test,
and removed original tests in ksck-test.
Change-Id: I1139bf9b3582ac9fa45bf341ec5b97f8048c92b0
---
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck_remote-test.cc
2 files changed, 230 insertions(+), 171 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/42/14142/4
--
To view, visit http://gerrit.cloudera.org:8080/14142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1139bf9b3582ac9fa45bf341ec5b97f8048c92b0
Gerrit-Change-Number: 14142
Gerrit-PatchSet: 4
Gerrit-Owner: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
