Jialun Peng created KAFKA-19612:
-----------------------------------

             Summary: Remove Redundant Validation in StripedReplicaPlacer
                 Key: KAFKA-19612
                 URL: https://issues.apache.org/jira/browse/KAFKA-19612
             Project: Kafka
          Issue Type: Improvement
            Reporter: Jialun Peng
            Assignee: Jialun Peng


Currently, the following validation checks are performed twice for each 
partition placement:
 # {{throwInvalidReplicationFactorIfNonPositive()}}

 # {{throwInvalidReplicationFactorIfTooFewBrokers()}}

 # {{throwInvalidReplicationFactorIfZero()}}

These checks are already performed in the public {{place()}} method before the 
partition placement loop begins. Since the cluster state and replication factor 
don't change during the placement operation, these checks only need to be 
performed once.

 

This redundant validation could be removed from the inner loop to improve 
performance, especially when placing many partitions.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to