divijvaidya commented on code in PR #14161:
URL: https://github.com/apache/kafka/pull/14161#discussion_r1288309828
##########
core/src/main/scala/kafka/server/ConfigHandler.scala:
##########
@@ -62,6 +62,12 @@ class TopicConfigHandler(private val logManager: LogManager,
kafkaConfig: KafkaC
topicConfig.asScala.forKeyValue { (key, value) =>
if (!configNamesToExclude.contains(key)) props.put(key, value)
}
+
+ if (!kafkaConfig.remoteLogManagerConfig.enableRemoteStorageSystem()
Review Comment:
> For my understanding, will this approach work if the user wants to
downgrade from 3.6 to 2.8?
Correct me if I am wrong here @kamalcph but downgrade is not supported in
Kafka as per "Once the brokers begin using the latest protocol version, it will
no longer be possible to downgrade the cluster to an older version." from
https://kafka.apache.org/documentation.html#upgrade_350_zk This is true
irrespective of Tiered Storage or not.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
