KAFKA-18408 tweak the 'tag' field for BrokerHeartbeatRequest.json, BrokerRegistrationChangeRecord.json and RegisterBrokerRecord.json (apache#18421)

mingdaoy · web-flow · commit c40cc5740f1e · 2025-01-08T04:16:59.000+08:00
Reviewers: Chia-Ping Tsai &lt;chia7712@gmail.com&gt;
diff --git a/clients/src/main/resources/common/message/BrokerHeartbeatRequest.json b/clients/src/main/resources/common/message/BrokerHeartbeatRequest.json
@@ -31,7 +31,7 @@
       "about": "True if the broker wants to be fenced, false otherwise." },
     { "name": "WantShutDown", "type": "bool", "versions": "0+",
       "about": "True if the broker wants to be shut down, false otherwise." },
-    { "name": "OfflineLogDirs", "type":  "[]uuid", "versions": "1+", "taggedVersions": "1+", "tag": "0",
+    { "name": "OfflineLogDirs", "type":  "[]uuid", "versions": "1+", "taggedVersions": "1+", "tag": 0,
       "about": "Log directories that failed and went offline." }
   ]
 }
diff --git a/metadata/src/main/resources/common/metadata/BrokerRegistrationChangeRecord.json b/metadata/src/main/resources/common/metadata/BrokerRegistrationChangeRecord.json
@@ -30,7 +30,7 @@
      "about": "-1 if the broker has been unfenced, 0 if no change, 1 if the broker has been fenced." },
    { "name": "InControlledShutdown", "type": "int8", "versions": "1+", "taggedVersions": "1+", "tag": 1,
      "about": "0 if no change, 1 if the broker is in controlled shutdown." },
-   { "name": "LogDirs", "type":  "[]uuid", "versions":  "2+", "taggedVersions": "2+", "tag": "2",
+   { "name": "LogDirs", "type":  "[]uuid", "versions":  "2+", "taggedVersions": "2+", "tag": 2,
      "about": "Log directories configured in this broker which are available." }
   ]
 }
diff --git a/metadata/src/main/resources/common/metadata/RegisterBrokerRecord.json b/metadata/src/main/resources/common/metadata/RegisterBrokerRecord.json
@@ -57,7 +57,7 @@
       "about": "True if the broker is fenced." },
     { "name": "InControlledShutdown", "type": "bool", "versions": "1+", "default": "false",
       "about": "True if the broker is in controlled shutdown." },
-    { "name": "LogDirs", "type":  "[]uuid", "versions":  "3+", "taggedVersions": "3+", "tag": "0",
+    { "name": "LogDirs", "type":  "[]uuid", "versions":  "3+", "taggedVersions": "3+", "tag": 0,
       "about": "Log directories configured in this broker which are available." }
   ]
 }

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@`
`31`	`31`	`"about": "True if the broker wants to be fenced, false otherwise." },`
`32`	`32`	`{ "name": "WantShutDown", "type": "bool", "versions": "0+",`
`33`	`33`	`"about": "True if the broker wants to be shut down, false otherwise." },`
`34`		`- { "name": "OfflineLogDirs", "type": "[]uuid", "versions": "1+", "taggedVersions": "1+", "tag": "0",`
	`34`	`+ { "name": "OfflineLogDirs", "type": "[]uuid", "versions": "1+", "taggedVersions": "1+", "tag": 0,`
`35`	`35`	`"about": "Log directories that failed and went offline." }`
`36`	`36`	`]`
`37`	`37`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@`
`30`	`30`	`"about": "-1 if the broker has been unfenced, 0 if no change, 1 if the broker has been fenced." },`
`31`	`31`	`{ "name": "InControlledShutdown", "type": "int8", "versions": "1+", "taggedVersions": "1+", "tag": 1,`
`32`	`32`	`"about": "0 if no change, 1 if the broker is in controlled shutdown." },`
`33`		`- { "name": "LogDirs", "type": "[]uuid", "versions": "2+", "taggedVersions": "2+", "tag": "2",`
	`33`	`+ { "name": "LogDirs", "type": "[]uuid", "versions": "2+", "taggedVersions": "2+", "tag": 2,`
`34`	`34`	`"about": "Log directories configured in this broker which are available." }`
`35`	`35`	`]`
`36`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@`
`57`	`57`	`"about": "True if the broker is fenced." },`
`58`	`58`	`{ "name": "InControlledShutdown", "type": "bool", "versions": "1+", "default": "false",`
`59`	`59`	`"about": "True if the broker is in controlled shutdown." },`
`60`		`- { "name": "LogDirs", "type": "[]uuid", "versions": "3+", "taggedVersions": "3+", "tag": "0",`
	`60`	`+ { "name": "LogDirs", "type": "[]uuid", "versions": "3+", "taggedVersions": "3+", "tag": 0,`
`61`	`61`	`"about": "Log directories configured in this broker which are available." }`
`62`	`62`	`]`
`63`	`63`	`}`