----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/ -----------------------------------------------------------
(Updated May 18, 2017, 8:57 p.m.) Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li, Sergio Pena, Vamsee Yarlagadda, and Vadim Spector. Changes ------- Description changed based on the new patch updated. Bugs: SENTRY-1736 https://issues.apache.org/jira/browse/SENTRY-1736 Repository: sentry Description (updated) ------- 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 ----- 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/3/ Testing ------- Made sure that Solr client is able to talk to sentry with kerberos enabled. Thanks, kalyan kumar kalvagadda