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

Prashanth Menon commented on KAFKA-295:
---------------------------------------

This is a small but important bug.  I can drop a patch since I've been touching 
that code anyways.  It will probably come in after KAFKA-49.
                
> Bug in async producer DefaultEventHandler retry logic
> -----------------------------------------------------
>
>                 Key: KAFKA-295
>                 URL: https://issues.apache.org/jira/browse/KAFKA-295
>             Project: Kafka
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 0.8
>            Reporter: Prashanth Menon
>            Assignee: Prashanth Menon
>             Fix For: 0.8
>
>
> In the DefaultEventHandler's retry loop, the logic should not return after a 
> successful retry.  Rather, it should set a boolean flag indicating that the 
> retry was successful and exit or break the while loop.  In the end it should 
> throw an exception only the flag is false.  Otherwise, it should continue the 
> outer for loop and send remaining data to remaning brokers.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to