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

Guozhang Wang commented on KAFKA-1740:
--------------------------------------

Thanks for the findings [~onurkaraman]! And sorry I overlooked this issue when 
making the commit.

As for the solution options, I actually still prefer option 1 because I feel it 
makes more sense to me to only log generation id increase when we finished form 
a new group so that we can log the new group members along with the new 
generation id (so I guess it is purely personal taste :)

Anyways I think this is an urgent bug fix, so if you are OK with option 1 I can 
extract it from [~hachikuji]'s patch and commit it to unblock people?

> Merge Offset manager into Coordinator
> -------------------------------------
>
>                 Key: KAFKA-1740
>                 URL: https://issues.apache.org/jira/browse/KAFKA-1740
>             Project: Kafka
>          Issue Type: Sub-task
>          Components: consumer
>            Reporter: Guozhang Wang
>            Assignee: Guozhang Wang
>            Priority: Critical
>             Fix For: 0.8.3
>
>         Attachments: KAFKA-1740.patch, KAFKA-1740_2015-06-29_18:21:42.patch, 
> KAFKA-1740_2015-06-29_18:44:54.patch
>
>
> This JIRA involves refactoring offset manager and merge it into coordinator, 
> including adding the logic for consumer-id / generation-id checking.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to