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

Reply via email to