Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23841 )

Change subject: [Java] Fix TLS truststore init for FIPS providers
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23841/1/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java
File java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java:

http://gerrit.cloudera.org:8080/#/c/23841/1/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java@420
PS1, Line 420: new char[0]
It seems this argument might be null as well at least for 
java.security.KeyStore, per the documentation page [1] in the description of 
this changelist.  Did you find null as the second parameter isn't accepted by 
some particular security provider, so an empty string is used instead?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I361341146c9115a17df182c397dbf4119ec68464
Gerrit-Change-Number: 23841
Gerrit-PatchSet: 1
Gerrit-Owner: Marton Greber <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Thu, 08 Jan 2026 17:48:27 +0000
Gerrit-HasComments: Yes

Reply via email to