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

Review request for Flume.


Summary
-------

1) Allows users to configure a restart flag and sleep interval for the 
ExecSource should the command exit.
2) Currently no one is reading standard error for the sub process. It is not 
ignored like the documentation suggests. A thread is added which by default 
reads and throws away stderr. If a flag is turned on it will logstderr. This 
kind of flag can be very useful when debugging why a command is not outputting 
the expected data.


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


Diffs
-----

  flume-ng-core/pom.xml d753fa1 
  flume-ng-core/src/main/java/org/apache/flume/source/ExecSource.java a96016c 
  flume-ng-core/src/test/java/org/apache/flume/source/TestExecSource.java 
3014368 

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


Testing
-------

A test for the restart flag and restartThrottle flag were added. All unit tests 
pass.


Thanks,

Brock

Reply via email to