[ 
https://issues.apache.org/jira/browse/KAFKA-2557?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Onur Karaman updated KAFKA-2557:
--------------------------------
    Description: 
The ILLEGAL_GENERATION error is a bit confusing today. When a consumer receives 
an ILLEGAL_GENERATION from hearbeat response, it should still use that 
generation id to commit offset. i.e. the generation id was not really illegal.

The current code was written earlier when we still bump up the generation id 
when the coordinator enters PrepareRebalance state. Since now the generation id 
is bumped up after PreareRebalance state ends, we should not overload 
ILLEGAL_GENERATION to notify a rebalance but create a new error code such as 
REBALANCE_IN_PROGRESS.

  was:
The IllegalGenerationIdException is a bit confusing today. When a consumer 
receives an IllegalGenerationIdException from hearbeat response, it should 
still use that generation id to commit offset. i.e. the generation ID was not 
really illegal. 

The current code was written earlier when we still bump up the generation ID 
when the coordinator enters PrepareRebalance state. Since now the generation ID 
is bumped up after PreareRebalance state ends, we should not overload 
IllegalGenerationIdException to notify a rebalance but create a new error code 
such as RebalanceNeededException.


> Separate RebalanceInProgress from IllegalGeneration Error Code
> --------------------------------------------------------------
>
>                 Key: KAFKA-2557
>                 URL: https://issues.apache.org/jira/browse/KAFKA-2557
>             Project: Kafka
>          Issue Type: Bug
>          Components: clients, consumer
>            Reporter: Jiangjie Qin
>            Assignee: Onur Karaman
>
> The ILLEGAL_GENERATION error is a bit confusing today. When a consumer 
> receives an ILLEGAL_GENERATION from hearbeat response, it should still use 
> that generation id to commit offset. i.e. the generation id was not really 
> illegal.
> The current code was written earlier when we still bump up the generation id 
> when the coordinator enters PrepareRebalance state. Since now the generation 
> id is bumped up after PreareRebalance state ends, we should not overload 
> ILLEGAL_GENERATION to notify a rebalance but create a new error code such as 
> REBALANCE_IN_PROGRESS.



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

Reply via email to