-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59317/
-----------------------------------------------------------

(Updated May 18, 2017, 12:06 a.m.)


Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li, 
Sergio Pena, Vamsee Yarlagadda, and Vadim Spector.


Changes
-------

Addressed review comments.


Bugs: SENTRY-1736
    https://issues.apache.org/jira/browse/SENTRY-1736


Repository: sentry


Description
-------

As part of SENTRY-1593, code in SentryGenericServiceClientDefaultImpl which set 
HADOOP_SECURITY_AUTHENTICATION property and update UserGroupInformation class 
is missed.


Diffs (updated)
-----

  
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryTransportFactory.java
 9b9f9e8 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClientDefaultImpl.java
 b7ac640 


Diff: https://reviews.apache.org/r/59317/diff/2/

Changes: https://reviews.apache.org/r/59317/diff/1-2/


Testing
-------

Made sure that Solr client is able to talk to sentry with kerberos enabled.


Thanks,

kalyan kumar kalvagadda

Reply via email to