zhouyifan279 commented on issue #1204: URL: https://github.com/apache/incubator-kyuubi/issues/1204#issuecomment-942116481
> It looks fine to me. However, can we merge some of these configurations, it is now very hard to explain and use. > > ``` > kyuubi.ha.zookeeper.acl.enabled=true > kyuubi.ha.zookeeper.auth.sasl.kerberos=false > kyuubi.ha.zookeeper.auth=digest:hue:*** > kyuubi.ha.zookeeper.acl.engine.enabled=true > ``` > > cc @zhouyifan279, do you have any idea, if we can add some unit tests to test with a kerberied zookeeper and acls As we already have `org.apache.kyuubi.KerberizedTestHelper` to setup a KDC, and zookeeper integrates kerberos using JAAS, it should be easy to setup an kerberied embedde zookeeper server. @wForget , would you mind to add these test cases ? I'm also glad to do the work if you have no time. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
