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

(Updated 2012-02-22 01:26:59.240182)


Review request for Flume.


Changes
-------

Updated diff based on feedback. Now if Sink throws a 
FlumePermanentFailureException the SinkRunner will terminate.


Summary
-------

Catches, logs, and sleeps when a general exception occurs. This is similar to 
PollingSourceRunner.


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


Diffs (updated)
-----

  
flume-ng-core/src/main/java/org/apache/flume/FlumePermanentFailureException.java
 PRE-CREATION 
  flume-ng-core/src/main/java/org/apache/flume/SinkRunner.java 9e46d87 

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


Testing
-------

All unit tests pass.


Thanks,

Brock

Reply via email to