fuyou001 opened a new pull request, #10820:
URL: https://github.com/apache/rocketmq/pull/10820

   ### Which Issue(s) This PR Fixes
   
   - Fixes #10819
   
   ### Brief Description
   
   Correct the misleading comment before `byteBuffer.put(preEncodeBuffer)` in 
`DefaultAppendMessageCallback#doAppend(...)`.
   
   The destination buffer belongs to the CommitLog append path and may be 
backed by the mapped byte buffer, the transient write buffer, or the shared 
staging buffer used by `writeWithoutMmap`. It is not a queue or ConsumeQueue 
buffer. The revised comment also makes clear that this line copies an already 
encoded message.
   
   This is a documentation-only change with no runtime behavior, API, protocol, 
configuration, or persisted-format impact.
   
   ### How Did You Test This Change?
   
   - Ran `git diff --check` successfully.
   - No behavioral tests were added or run because only a source-code comment 
changed.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to