Attila Bukor 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 2:

(1 comment)

> 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 set
Yep, thanks for the suggestion.



--
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: 2
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 24 Aug 2021 06:41:00 +0000
Gerrit-HasComments: Yes

Reply via email to