> On June 25, 2012, 7:54 a.m., Mubarak Seyed wrote: > > flume-ng-core/src/main/java/org/apache/flume/sink/NullSink.java, line 91 > > <https://reviews.apache.org/r/5545/diff/2/?file=116192#file116192line91> > > > > Don't we need to count the success counts here? Null sink may not have > > to but client app needs to do math like (generated - successful) = > > dropped_events > > Hari Shreedharan wrote: > While this is not impossible to do, I don't want to introduce extra > complex code for this calculation. I will submit a patch with the rest of the > concerns addressed.
Addressed this in the latest patch. - Hari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5545/#review8533 ----------------------------------------------------------- On June 25, 2012, 6:20 p.m., Hari Shreedharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5545/ > ----------------------------------------------------------- > > (Updated June 25, 2012, 6:20 p.m.) > > > Review request for Flume. > > > Description > ------- > > Make Null sink configurable. > > > This addresses bug FLUME-1315. > https://issues.apache.org/jira/browse/FLUME-1315 > > > Diffs > ----- > > > flume-ng-channels/flume-recoverable-memory-channel/src/test/java/org/apache/flume/channel/recoverable/memory/TestRecoverableMemoryChannel.java > 6e0ec2b > > flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkConfiguration.java > 0a91436 > flume-ng-core/src/main/java/org/apache/flume/sink/NullSink.java c812851 > > flume-ng-node/src/test/java/org/apache/flume/node/TestAbstractLogicalNodeManager.java > 521b586 > > Diff: https://reviews.apache.org/r/5545/diff/ > > > Testing > ------- > > > Thanks, > > Hari Shreedharan > >
