Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17786 )
Change subject: [spark] KUDU-1921 Add ability to require authn/encryption ...................................................................... Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/17786/1/java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala File java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala: http://gerrit.cloudera.org:8080/#/c/17786/1/java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala@951 PS1, Line 951: } Does it make sense to add a scenario when a client with the the default settings tries to connect and succeeds when the cluster is run with --rpc_encryption=disabled --rpc_authentication=disabled? That would validate the default settings at the client side and make sure the Kudu client in kudu-spark package behaves as expected. -- To view, visit http://gerrit.cloudera.org:8080/17786 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iba1877e13a3218f0c285ded6c0f7047c497ef6aa Gerrit-Change-Number: 17786 Gerrit-PatchSet: 1 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 20 Aug 2021 02:23:28 +0000 Gerrit-HasComments: Yes
