> On Sept. 24, 2012, 6:11 p.m., Brock Noland wrote:
> > flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/FileChannel.java,
> >  line 404
> > <https://reviews.apache.org/r/7228/diff/1/?file=159481#file159481line404>
> >
> >     Cool! I think that some of the unit tests look for this error message. 
> > Have you ran the file channel unit tests to see which ones failed?

Newbees mistake.  give me a couple of minutes


- Ted


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


On Sept. 24, 2012, 6:09 p.m., Ted Malaska wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7228/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2012, 6:09 p.m.)
> 
> 
> Review request for Flume.
> 
> 
> Description
> -------
> 
> Updated old exception message to match that of Brock Noland.
> 
> 
> This addresses bug FLUME-1603.
>     https://issues.apache.org/jira/browse/FLUME-1603
> 
> 
> Diffs
> -----
> 
>   
> flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/FileChannel.java
>  c2e904b 
> 
> Diff: https://reviews.apache.org/r/7228/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ted Malaska
> 
>

Reply via email to