fvaleri commented on code in PR #20291:
URL: https://github.com/apache/kafka/pull/20291#discussion_r2249246490


##########
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 metadata for each record and control records.")

Review Comment:
   We skip the entire control record, both data and metadata, but I rephrased 
similar to what you suggest. Thanks.



-- 
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

Reply via email to