Hello Dan Burkert, Grant Henke,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/11325

to review the following change.


Change subject: TestSecurity: remove errant Precondition
......................................................................

TestSecurity: remove errant Precondition

Its presence makes it impossible to retry the tests in this class.

An alternative would be to null out miniCluster in tearDown(), but I don't
think that should be necessary for creating a new cluster.

Change-Id: I0ba826d3ec722cb20b35d4c34817d30ab9782976
---
M java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/25/11325/1
--
To view, visit http://gerrit.cloudera.org:8080/11325
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0ba826d3ec722cb20b35d4c34817d30ab9782976
Gerrit-Change-Number: 11325
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>

Reply via email to