yx9o commented on PR #10287:
URL: https://github.com/apache/rocketmq/pull/10287#issuecomment-4412579210

   > @yx9o @zhishengzhang
   
   Sorry for the late reply.
   
   Just to confirm, this PR and #10283 are both trying to address #10284, but 
the approaches look different. #10283 changes the generic RemotingSerializable 
decode path, while this PR narrows the fix to the ConsumerOffset response shape.
   
   If we keep this approach, could we add a regression test covering the mixed 
offsetTable + pullOffsetTable response returned by GET_ALL_CONSUMER_OFFSET?
   


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