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