[ 
https://issues.apache.org/jira/browse/FLUME-762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13103936#comment-13103936
 ] 

[email protected] commented on FLUME-762:
-----------------------------------------------------



bq.  On 2011-09-13 19:55:07, jmhsieh wrote:
bq.  > Applied patch, these unit tests pass.  However, to cases broke:
bq.  > 
bq.  > com.cloudera.flume.agent.TestFlumeNode.testRestartNextException
bq.  > com.cloudera.flume.agent.diskfailover.TestDiskFailoverDeco.testHandleExns
bq.  > 
bq.  > Please fix.

Rephrase: The unit test included pass, but the  code broke two others (which I 
think are related)


- jmhsieh


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


On 2011-09-12 17:39:06, Prasad Mujumdar wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1739/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-09-12 17:39:06)
bq.  
bq.  
bq.  Review request for jmhsieh.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  If the source or sink throws an exception, close, reopen and retry it. 
This way the flow can continue after minor/recoverable errors.
bq.  
bq.  
bq.  This addresses bug FLUME-762.
bq.      https://issues.apache.org/jira/browse/FLUME-762
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    
flume-core/src/main/java/com/cloudera/flume/core/connector/DirectDriver.java 
0466394 
bq.    
flume-core/src/test/java/com/cloudera/flume/core/connector/TestDirectDriverExp.java
 PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/1739/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Prasad
bq.  
bq.



> DirectDriver should reopen the source/sink if the append or next throws an 
> exception
> ------------------------------------------------------------------------------------
>
>                 Key: FLUME-762
>                 URL: https://issues.apache.org/jira/browse/FLUME-762
>             Project: Flume
>          Issue Type: Improvement
>          Components: Node
>            Reporter: Prasad Mujumdar
>            Assignee: Prasad Mujumdar
>         Attachments: flume-762.patch
>
>
> Currently, if source or sink throws an exception, the driver bails out 
> resulting the everything to shut down. Alternately, we can try to close and 
> reopen the source or sink, and then attempt to continue. This way we can 
> avoid blocking the flow. If the reopen fails, then it can bail out.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to