You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by cm...@apache.org on 2024/03/27 19:34:07 UTC
(kafka) branch 3.6 updated: KAFKA-16428: Fix bug where config change notification znode may not get created during migration (#15608)
This is an automated email from the ASF dual-hosted git repository.
cmccabe pushed a commit to branch 3.6
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/3.6 by this push:
new 7303b6063bc KAFKA-16428: Fix bug where config change notification znode may not get created during migration (#15608)
7303b6063bc is described below
commit 7303b6063bc16a4bc8a75083a55460be08c5c9c0
Author: Colin Patrick McCabe <cm...@apache.org>
AuthorDate: Wed Mar 27 12:26:36 2024 -0700
KAFKA-16428: Fix bug where config change notification znode may not get created during migration (#15608)
Reviewers: Chia-Ping Tsai <ch...@gmail.com>, Luke Chen <sh...@gmail.com>
---
core/src/main/scala/kafka/zk/migration/ZkConfigMigrationClient.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/core/src/main/scala/kafka/zk/migration/ZkConfigMigrationClient.scala b/core/src/main/scala/kafka/zk/migration/ZkConfigMigrationClient.scala
index 844c1aabc4c..fbf6d7c3434 100644
--- a/core/src/main/scala/kafka/zk/migration/ZkConfigMigrationClient.scala
+++ b/core/src/main/scala/kafka/zk/migration/ZkConfigMigrationClient.scala
@@ -244,7 +244,7 @@ class ZkConfigMigrationClient(
state
} else if (responses.head.resultCode.equals(Code.OK)) {
// Write the notification znode if our update was successful
- zkClient.createConfigChangeNotification(s"$configType/$configName")
+ zkClient.createConfigChangeNotification(s"${configType.get}/$configName")
state.withMigrationZkVersion(migrationZkVersion)
} else {
throw KeeperException.create(responses.head.resultCode, path)