cmccabe commented on a change in pull request #10812:
URL: https://github.com/apache/kafka/pull/10812#discussion_r658327187
##########
File path: core/src/main/scala/kafka/server/KafkaConfig.scala
##########
@@ -374,6 +375,7 @@ object KafkaConfig {
val BrokerSessionTimeoutMsProp = "broker.session.timeout.ms"
val NodeIdProp = "node.id"
val MetadataLogDirProp = "metadata.log.dir"
+ val MetadataSnapshotMinNewRecordBytesProp =
"metadata.log.snapshot.min.new_record.bytes"
Review comment:
Hmm, I think we should avoid underscores in configuration key names.
Also, min.new_record.bytes is a bit unclear (to me at least).
How about `metadata.log.min.record.bytes.between.snapshots` ?
--
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
