Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17733 )

Change subject: [python] KUDU-1921 Add ability to require auth/encryption
......................................................................


Patch Set 9:

> Patch Set 8: Code-Review+1
>
> Looks good to me overall.
>
> Is it possible to add a small test scenario to cover the newly introduced 
> flags, maybe just for non-kerberized case?  It seems there is mini-cluster 
> test scaffolding in python/kudu/tests/common.py.

Added some tests (which was a very good idea indeed, had to fix the code to).


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I10173145611ad2991c0a1b173ecadc7141ae6f5e
Gerrit-Change-Number: 17733
Gerrit-PatchSet: 9
Gerrit-Owner: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 17 Aug 2021 15:46:52 +0000
Gerrit-HasComments: No

Reply via email to