hachikuji commented on a change in pull request #10256: URL: https://github.com/apache/kafka/pull/10256#discussion_r587803031
########## File path: core/src/main/scala/kafka/raft/RaftManager.scala ########## @@ -220,25 +220,14 @@ class KafkaRaftManager[T]( } private def buildMetadataLog(): KafkaMetadataLog = { - val defaultProps = LogConfig.extractLogConfigMap(config) Review comment: It seemed best for now to not allow overrides since this is a new usage of `Log` and we haven't had time to understand the impact of all of the configurations for this usage. Most of them are probably safe, but others do not even make sense (e.g. the retention settings). Let me open a JIRA so that we can consider which configs we want to expose for the metadata log and how we want to expose them. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org