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

2020-12-18 Thread wangning (Code Review)
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 <1994wangn...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] [doc] add a note about KUDU-3217 into known issues

2020-12-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16889 )

Change subject: [doc] add a note about KUDU-3217 into known issues
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2a8fe02e337d62539d30ae4f22377e150b344392
Gerrit-Change-Number: 16889
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Greg Solovyev 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Liliana Kadar 
Gerrit-Comment-Date: Sat, 19 Dec 2020 03:39:59 +
Gerrit-HasComments: No


[kudu-CR] [doc] add a note about KUDU-3217 into known issues

2020-12-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16889


Change subject: [doc] add a note about KUDU-3217 into known issues
..

[doc] add a note about KUDU-3217 into known issues

Change-Id: I2a8fe02e337d62539d30ae4f22377e150b344392
---
M docs/known_issues.adoc
1 file changed, 7 insertions(+), 0 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2a8fe02e337d62539d30ae4f22377e150b344392
Gerrit-Change-Number: 16889
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 


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

2020-12-18 Thread wangning (Code Review)
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 (#2).

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

[java] allow disable TLS transportation from java client

In a cluster which is running in intranet, security is less concerned than
perfromance cause port is not exposed. And to disable TLS from server side
requires to reboot cluster, it's a big burden for a working cluster.
Also, on JAVA8 the TLS cipher perfromance is poor[1].

This commit introduced DISABLE_KUDU_CLIENT_TLS environment to allow
disable TLS transportation when talking to server.

[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, 108 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/16887/2
--
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: 2
Gerrit-Owner: wangning <1994wangn...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)


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

2020-12-18 Thread wangning (Code Review)
wangning has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/16887


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

[java] allow disable TLS transportation from java client

In a cluster which is running in intranet, security is less concerned than
perfromance cause port is not exposed. And to disable TLS from server side
requires to reboot cluster, it's a big burden for a working cluster.
Also, on JAVA8 the TLS cipher perfromance is poor[1].

This commit introduced DISABLE_KUDU_CLIENT_TLS environment to allow
disable TLS transportation when talking to server.

[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
1 file changed, 15 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/16887/1
--
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: newchange
Gerrit-Change-Id: I762b9d6ef9257a7853684dd0aef315e5c5bba8ed
Gerrit-Change-Number: 16887
Gerrit-PatchSet: 1
Gerrit-Owner: wangning <1994wangn...@gmail.com>