You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "wangning (Code Review)" <ge...@cloudera.org> on 2020/12/19 03:45:36 UTC

[kudu-CR] [java] allow disabling TLS transportation from java client

Hello Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#3).

Change subject: [java] allow disabling TLS transportation from java client
......................................................................

[java] allow disabling TLS transportation from java client

In a cluster which is running in intranet, sometimes security is less concerned
than performance in case port is not exposed.
As mentioned in [1], on JAVA8 the TLS cypher performance is poor.
And to disable TLS from server side requires to reboot cluster,
it worths the trade off for a working cluster.

This commit introduce DISABLE_KUDU_CLIENT_TLS environment to allow
disabling use TLS transportation in client side.

[1] https://issues.apache.org/jira/browse/KUDU-3133

Change-Id: I762b9d6ef9257a7853684dd0aef315e5c5bba8ed
---
M java/kudu-client/src/main/java/org/apache/kudu/client/Negotiator.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiation.java
2 files changed, 109 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/16887/3
-- 
To view, visit http://gerrit.cloudera.org:8080/16887
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I762b9d6ef9257a7853684dd0aef315e5c5bba8ed
Gerrit-Change-Number: 16887
Gerrit-PatchSet: 3
Gerrit-Owner: wangning <19...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)