Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Flume >> mail # dev >> Review Request: FLUME-1490: Option to limit number of events sent in Stress source


Copy link to this message
-
Re: Review Request: FLUME-1490: Option to limit number of events sent in Stress source

-----------------------------------------------------------
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