-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59317/
-----------------------------------------------------------
(Updated May 19, 2017, 1:18 a.m.)
Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li,
Sergio Pena, Vamsee Yarlagadda, and Vadim Spector.
Changes
-------
Made changes to avaoid issues that might happen when multiple threads try to
call initialize() initially at the same time. one of the threads might return
with out initializing UserGroupInformation.
Bugs: SENTRY-1736
https://issues.apache.org/jira/browse/SENTRY-1736
Repository: sentry
Description
-------
1. Update the UserGroupInformation with HADOOP_SECURITY_AUTHENTICATION for the
client when kerberos is enabled.
2. Make sure that the update is not done for every connection towards sentry
server.
3. Don't update the configuration object that we passed. Instead use a local
version to update.
Diffs (updated)
-----
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryTransportFactory.java
9b9f9e8
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/UserGroupInformationInitializer.java
PRE-CREATION
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/5/
Changes: https://reviews.apache.org/r/59317/diff/4-5/
Testing
-------
Made sure that Solr client is able to talk to sentry with kerberos enabled.
Thanks,
kalyan kumar kalvagadda
