[ https://issues.apache.org/jira/browse/HIVE-22195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16928307#comment-16928307 ]
Peter Vary commented on HIVE-22195: ----------------------------------- [~dkuzmenko]: Wouldn't be more readable if we create a method in ZooKeeperUtil.isSecurityEnabled, and use that in every place? And maybe add some test to the util method at least? Otherwise LGTM > Configure authentication type for Zookeeper when different from the default > cluster wide > ---------------------------------------------------------------------------------------- > > Key: HIVE-22195 > URL: https://issues.apache.org/jira/browse/HIVE-22195 > Project: Hive > Issue Type: Improvement > Reporter: Denys Kuzmenko > Assignee: Denys Kuzmenko > Priority: Major > Attachments: HIVE-22195.1.patch > > > This could be useful in case when cluster is kerberized, but Zookeeper is not. -- This message was sent by Atlassian Jira (v8.3.2#803003)