----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16025/#review29813 -----------------------------------------------------------
Ship it! Ship It! - Brock Noland On Dec. 5, 2013, 12:43 a.m., Gregory Chanan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16025/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2013, 12:43 a.m.) > > > Review request for sentry, Brock Noland and Shreepadma Venugopalan. > > > Repository: sentry > > > Description > ------- > > solr lazily authenticates with HDFS, and there can be a race between when > solr authenticates and when sentry would try to read the .ini file off of > HDFS. > > I don't think this is an issue with sentry and hive; AFAIK hive will already > be authenticated by the time sentry runs in that case. > > This patch gives the solr sentry binding the ability to authenticate via > kerberos. > > > Diffs > ----- > > > sentry-binding/sentry-binding-solr/src/main/java/org/apache/sentry/binding/solr/authz/SolrAuthzBinding.java > 5c2d7bc > > Diff: https://reviews.apache.org/r/16025/diff/ > > > Testing > ------- > > Ran the unit tests successfully and tried the new sentry on a solr cluster > and it worked. > > > Thanks, > > Gregory Chanan > >
