[
https://issues.apache.org/jira/browse/KAFKA-13160?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ryan Dielhenn updated KAFKA-13160:
----------------------------------
Description:
In a ZK cluster, dynamic default broker configs are stored in the zNode
/brokers/<default>. Without this fix, when dynamic configs from snapshots are
processed by the brokers in KRaft, the BrokerConfigHandler checks if the
resource name is "<default>" to do a default update and converts the resource
name to an integer otherwise to do a per-broker config update.
In KRaft, dynamic default broker configs are serialized in the quorum with
empty string instead of "<default>". This was causing the BrokerConfigHandler
to throw a NumberFormatException for dynamic default broker configs since the
resource name for them is not "<default>" or a single integer. This handler
should be fixed to expect empty string as the resource name for the dynamic
default broker configs if using KRaft.
was:
In a ZK cluster, dynamic default broker configs are stored in the zNode
/brokers/<default>. Without this fix, when dynamic configs from snapshots are
processed by the brokers in KRaft, the BrokerConfigHandler checks if the
resource name is "<default>" to do a default update and converts the resource
name to an integer otherwise to do a per-broker config update.
In KRaft dynamic default broker configs are serialized in the quorum with empty
string instead of "<default>". This was causing the BrokerConfigHandler to
throw a NumberFormatException for dynamic default broker configs since the
resource name for them is not "<default>" or a single integer. This handler
should be fixed to expect empty string as the resource name for the dynamic
default broker configs if using KRaft.
> Fix BrokerConfigHandler to expect empty string as the resource name for
> dynamic default broker configs in KRaft
> ---------------------------------------------------------------------------------------------------------------
>
> Key: KAFKA-13160
> URL: https://issues.apache.org/jira/browse/KAFKA-13160
> Project: Kafka
> Issue Type: Bug
> Affects Versions: 3.0.0
> Reporter: Ryan Dielhenn
> Assignee: Ryan Dielhenn
> Priority: Blocker
> Fix For: 3.0.0
>
>
> In a ZK cluster, dynamic default broker configs are stored in the zNode
> /brokers/<default>. Without this fix, when dynamic configs from snapshots are
> processed by the brokers in KRaft, the BrokerConfigHandler checks if the
> resource name is "<default>" to do a default update and converts the resource
> name to an integer otherwise to do a per-broker config update.
> In KRaft, dynamic default broker configs are serialized in the quorum with
> empty string instead of "<default>". This was causing the BrokerConfigHandler
> to throw a NumberFormatException for dynamic default broker configs since the
> resource name for them is not "<default>" or a single integer. This handler
> should be fixed to expect empty string as the resource name for the dynamic
> default broker configs if using KRaft.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
