-----------------------------------------------------------
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

Reply via email to