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

Prashanth Menon commented on KAFKA-304:
---------------------------------------

Hi Neha,

I took a glancing look at the test and I think there might be a mistake.  The 
line that reads: "assertEquals(message, messageSet.next.message)" should 
actually be "assertEquals(message, messageAndOffset.message)".  The former 
progresses the iterator a second time after the preceeding line so two messages 
are consumed within one iteration of the loop, wherare the latter looks at one 
message within the current iteration alone.  I'll take a closer look when I get 
a chance.



                
> Simple Consumer message set behavior broken on 0.8 branch
> ---------------------------------------------------------
>
>                 Key: KAFKA-304
>                 URL: https://issues.apache.org/jira/browse/KAFKA-304
>             Project: Kafka
>          Issue Type: Bug
>    Affects Versions: 0.8
>            Reporter: Neha Narkhede
>            Assignee: Prashanth Menon
>            Priority: Critical
>         Attachments: kafka-240-bug.patch
>
>
> The testZKSendWithDeadBroker() test commented out in ProducerTest actually 
> exposes a bug with path KAFKA-240. It seems that even when the broker has 
> some messages for a topic partition, the simple consumer doesn't return those.
> I've simplified this test and attached a diff to this bug. Apply it on 0.8 
> branch to reproduce this. 

--
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