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

Review request for Flume and Arvind Prabhakar.


Description
-------

ChannelProcessor is eating exceptions. Here we catch Throwable to address the 
issue.

Also added in a small fix for FileChannel so it displays the name properly in 
the logs.


This addresses bug FLUME-1322.
    https://issues.apache.org/jira/browse/FLUME-1322


Diffs
-----

  
flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/FileChannel.java
 c703654 
  flume-ng-core/src/main/java/org/apache/flume/channel/ChannelProcessor.java 
42991de 

Diff: https://reviews.apache.org/r/5582/diff/


Testing
-------

Unit pass. Also, manual testing that an OutOfMemoryError gets printed to the 
logs in the middle of a Transaction (would not be printed before). Also checked 
that the FileChannel name gets printed to the logs during startup.


Thanks,

Mike Percy

Reply via email to