> On 2012-05-18 18:31:53, Hari Shreedharan wrote: > > Mike - It is better not to provide HbaseSink compiled against hadoop-23, > > since it will fail at run time(on hadoop-23), due to hadoop's compile only > > compatibility. So as part of FLUME-1208, I have removed Hbase sink compile > > itself in hadoop-23 profile. Please take a look at HBASE-6020. Until that > > is resolved, there is no point in building the Hbase sink against hadoop-23.
I see. Yeah I just wanted the build to compile. Thanks for looking at this Hari! I'll update the patch. - Mike ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5163/#review7977 ----------------------------------------------------------- On 2012-05-18 18:55:41, Mike Percy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5163/ > ----------------------------------------------------------- > > (Updated 2012-05-18 18:55:41) > > > Review request for Flume. > > > Summary > ------- > > Support for acting as multiple proxy users from a single, authenticated > principal. > > > This addresses bug FLUME-1196. > https://issues.apache.org/jira/browse/FLUME-1196 > > > Diffs > ----- > > > flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java > PRE-CREATION > > flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/KerberosUser.java > PRE-CREATION > > flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSFormatterFactory.java > d7963b1 > flume-ng-sinks/flume-hdfs-sink/pom.xml 3277a00 > > flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSEventSink.java > 8ce2bac > flume-ng-sinks/flume-hdfs-sink/src/test/resources/log4j.properties 361158d > flume-ng-sinks/flume-ng-hbase-sink/pom.xml ed2e44a > pom.xml 7a97c34 > > Diff: https://reviews.apache.org/r/5163/diff > > > Testing > ------- > > Added unit test and small framework for testing using MiniDFSCluster. > Existing tests pass. Also, a non-trivial amount of manual testing (no > Kerberos, Kerb only, Kerb+proxy, combination of Kerb & Kerb+proxy in same > agent) > > > Thanks, > > Mike > >
