-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35741/
-----------------------------------------------------------
Review request for sentry, Dapeng Sun and Sravya Tirukkovalur.
Bugs: SENTRY-776
https://issues.apache.org/jira/browse/SENTRY-776
Repository: sentry
Description
-------
Currently the Sentry client requires principal/keytab for ZK connection. Given
that it's a common client for various downstream project, we should also
support ticket cache based connect to ZK. The patch supports both options for
creating JAAS context for secure ZK connection. The method can be specified by
a Sentry config property.
Also fixed the missing default for RPC address property resolving _HOST part of
the kerberos principal.
Diffs
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/HAContext.java
71935b1
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/JaasConfiguration.java
d5f55fe
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
54dbac5
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java
1b9691e
Diff: https://reviews.apache.org/r/35741/diff/
Testing
-------
Thanks,
Prasad Mujumdar