-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7059/#review11426
-----------------------------------------------------------



flume-ng-core/src/main/java/org/apache/flume/source/StressSource.java
<https://reviews.apache.org/r/7059/#comment24460>

    subList doesn't modify the current list, it returns a view of the list with 
the requested elements


- Brock Noland


On Sept. 12, 2012, 6:34 p.m., Ted Malaska wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7059/
> -----------------------------------------------------------
> 
> (Updated Sept. 12, 2012, 6:34 p.m.)
> 
> 
> Review request for Flume.
> 
> 
> Description
> -------
> 
> If maxTotalEvents is -1 then don't ask how many are left.
> Also added a junit that tests the case where maxTotalEvents is not defined in 
> the config.
> 
> 
> This addresses bug FLUME-1570.
>     https://issues.apache.org/jira/browse/FLUME-1570
> 
> 
> Diffs
> -----
> 
>   flume-ng-core/src/main/java/org/apache/flume/source/StressSource.java 
> b4a00f5 
>   flume-ng-core/src/test/java/org/apache/flume/source/TestStressSource.java 
> e98a46f 
> 
> Diff: https://reviews.apache.org/r/7059/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ted Malaska
> 
>

Reply via email to