wang-jiahua opened a new issue, #10534:
URL: https://github.com/apache/rocketmq/issues/10534

   ### Before Creating the Enhancement Request
   
   - [x] I have confirmed that this should be classified as an enhancement 
rather than a bug/feature.
   
   ### Summary
   
   Implement `FastCodesHeader.encode()/decode()` for `PullMessageRequestHeader` 
and `PullMessageResponseHeader` to eliminate reflection-based header 
serialization.
   
   ### Motivation
   
   `PullMessageRequestHeader` and `PullMessageResponseHeader` already implement 
`FastCodesHeader` but their `decode()` methods use `Map<String, String>` 
instead of `HashMap<String, String>`, causing compilation issues with the 
interface. Additionally, their `encode()` methods can leverage R1's 
`writeLong`/`writeInt` helpers for numeric fields.
   
   ### Describe the Solution You'd Like
   
   Fix `decode()` signatures and optimize `encode()` methods on:
   - `PullMessageRequestHeader` — 15 fields including Long/Integer types
   - `PullMessageResponseHeader` — 4 fields
   
   ### Additional Context
   
   Depends on R1 (#10522, merged). Related PRs: #10443, #10526.
   


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