----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2660/#review2992 -----------------------------------------------------------
Ship it! - Prasad On 2011-11-01 08:15:52, Eric Sammer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2660/ > ----------------------------------------------------------- > > (Updated 2011-11-01 08:15:52) > > > Review request for Flume and Prasad Mujumdar. > > > Summary > ------- > > * Remove Hadoop as a flume-ng-core dep. > * Added commons-cli as an explicit dep. We were accidentally getting it via > hadoop. > * Cleaned up unused org.apache.hadoop.* class usage in the ExecSource test. > > > This addresses bug FLUME-825. > https://issues.apache.org/jira/browse/FLUME-825 > > > Diffs > ----- > > i/flume-ng-core/pom.xml 90dbcaa > i/flume-ng-core/src/test/java/org/apache/flume/source/TestExecSource.java > 6607bc1 > > Diff: https://reviews.apache.org/r/2660/diff > > > Testing > ------- > > Unit tests pass. > > > Thanks, > > Eric > >
