-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6673/
-----------------------------------------------------------
(Updated Aug. 17, 2012, 3:39 p.m.)
Review request for Flume.
Changes
-------
This new diff uses reflection to avoid relaxing visibility in StressSource's
event field. It also cleans up some extra scaffolding in the test file that
wasn't actually used.
Description
-------
This patch implements a count limit, both for overall (successful + failed)
events and for successful events. Seems like both could be useful in testing.
This addresses bug FLUME-1490.
https://issues.apache.org/jira/browse/FLUME-1490
Diffs (updated)
-----
flume-ng-core/pom.xml 8708953
flume-ng-core/src/main/java/org/apache/flume/source/StressSource.java 4f7b255
flume-ng-core/src/test/java/org/apache/flume/source/TestSequenceGeneratorSource.java
579b257
flume-ng-core/src/test/java/org/apache/flume/source/TestStressSource.java
PRE-CREATION
Diff: https://reviews.apache.org/r/6673/diff/
Testing
-------
Includes unit test and a bit of code clean-up.
Thanks,
Patrick Wendell
