Alexey Serbin has posted comments on this change. Change subject: [java] Reuse JVM across tests ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7825/1/java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduContext.scala File java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduContext.scala: Line 350: def clear(): Unit = { > Having cached connection that outlived the client was causing issues when w Interesting. Are those issues related only to the TLS certs or it's something else? As I understand, in the real-world scenarios, update on TLS certificates is supposed to be a very rare occasion: it might be expiration of the Kudu's IPKI certs (in 10 years after generation) or some case of re-generating those in case of security concerns. Anyway, I think clearing the cache might be good at least because not doing so might hide some issues with initial cache bootstrap for some corner cases. -- To view, visit http://gerrit.cloudera.org:8080/7825 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8e4c6d53054d5da4f2ecf9f4130a51df13d4fa6c Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes