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

Luke Chen updated KAFKA-12654:
------------------------------
    Description: Currently, when entering sticky assignor, we'll check if all 
consumers have the same subscription to decide which assignor to use 
(constrained assignor or general assignor). While checking the subscription, we 
also deserialize the subscription user data to get the partitions owned by the 
consumer (consumerToOwnedPartitions). However, the consumerToOwnedPartitions 
info is not used in general assignor, we don't need to deserialize data for 
general assignor. We should separate these 2 things into 2 methods, and only 
deserialize the user data for constrained assignor, to improve the general 
sticky assignor performance.  (was: Currently, when entering sticky assignor, 
we'll check if all consumers have the same subscription to decide which 
assignor to use (constrained assignor or general assignor). While checking the 
subscription, we also deserialize the subscription user data to get the 
partitions owned by the consumer (consumerToOwnedPartitions). However, the 
consumerToOwnedPartitions info is not used in general assignor. We should 
separate these 2 things into 2 methods, and only deserialize the user data for 
constrained assignor, to improve the general sticky assignor performance.)

> separate checkAllSubscriptionEqual and getConsumerToOwnedPartitions methods
> ---------------------------------------------------------------------------
>
>                 Key: KAFKA-12654
>                 URL: https://issues.apache.org/jira/browse/KAFKA-12654
>             Project: Kafka
>          Issue Type: Improvement
>            Reporter: Luke Chen
>            Assignee: Luke Chen
>            Priority: Major
>
> Currently, when entering sticky assignor, we'll check if all consumers have 
> the same subscription to decide which assignor to use (constrained assignor 
> or general assignor). While checking the subscription, we also deserialize 
> the subscription user data to get the partitions owned by the consumer 
> (consumerToOwnedPartitions). However, the consumerToOwnedPartitions info is 
> not used in general assignor, we don't need to deserialize data for general 
> assignor. We should separate these 2 things into 2 methods, and only 
> deserialize the user data for constrained assignor, to improve the general 
> sticky assignor performance.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to