----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9212/#review15986 -----------------------------------------------------------
Ship it! +1 lgtm - Will McQueen On Feb. 1, 2013, 3:41 a.m., Hari Shreedharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9212/ > ----------------------------------------------------------- > > (Updated Feb. 1, 2013, 3:41 a.m.) > > > Review request for Flume. > > > Description > ------- > > Set the keytab and principal in the conf object. > > > This addresses bug FLUME-1890. > https://issues.apache.org/jira/browse/FLUME-1890 > > > Diffs > ----- > > > flume-ng-sinks/flume-ng-hbase-sink/src/main/java/org/apache/flume/sink/hbase/HBaseSinkSecurityManager.java > 8bf0509 > > Diff: https://reviews.apache.org/r/9212/diff/ > > > Testing > ------- > > Ran on secure cluster and verified. > > > Thanks, > > Hari Shreedharan > >
