Pankraz76 commented on code in PR #20291: URL: https://github.com/apache/kafka/pull/20291#discussion_r2250055478
########## core/src/main/scala/kafka/tools/DumpLogSegments.scala: ########## @@ -638,7 +638,7 @@ object DumpLogSegments { " Instead, the value-decoder-class option can be used if a custom RLMM implementation is configured.") private val shareStateOpt = parser.accepts("share-group-state-decoder", "If set, log data will be parsed as share group state data from the " + "__share_group_state topic.") - private val skipRecordMetadataOpt = parser.accepts("skip-record-metadata", "Whether to skip printing metadata for each record.") + private val skipRecordMetadataOpt = parser.accepts("skip-record-metadata", "Whether to skip printing control records and metadata for each other record.") Review Comment: ```suggestion private val skipRecordMetadataOpt = parser.accepts("skip-record-metadata", "Whether to skip printing `control records`, and metadata for each other record.") ``` might highlight this as well. I mistaken it for control(s), records, and metadata, but control records is on term. -- 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