You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Hao Hao (Code Review)" <ge...@cloudera.org> on 2018/03/02 23:18:59 UTC

[kudu-CR] [java] fix the regression test for KUDU-2267/KUDU-2319

Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9388 )

Change subject: [java] fix the regression test for KUDU-2267/KUDU-2319
......................................................................

[java] fix the regression test for KUDU-2267/KUDU-2319

In the previous commit 0f0e42144, a regression test was added to ensure
a client with valid authn token but without valid Kerberos credentials
should be able to connect to all the masters. However, the test was not
correct. This patch fixes that.

Change-Id: I5e827586fe549f6a0c927ce8a4f8eca954bfe690
Reviewed-on: http://gerrit.cloudera.org:8080/9388
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
M java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
2 files changed, 22 insertions(+), 8 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5e827586fe549f6a0c927ce8a4f8eca954bfe690
Gerrit-Change-Number: 9388
Gerrit-PatchSet: 3
Gerrit-Owner: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins