You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/07/29 15:50:49 UTC

[GitHub] [pulsar] Gleiphir2769 commented on pull request #16797: [fix][client] Fix incorrect out-of-order check for chunked message in consumerImpl

Gleiphir2769 commented on PR #16797:
URL: https://github.com/apache/pulsar/pull/16797#issuecomment-1199591594

   > Could you please add a test to cover the new change?
   
   Hi, this out-of-bounds check will not be triggered in the correct client implementation. It will only be triggered if the producer client implement does not set the TotalChunkMsgSize correctly. So I think it's not neccessary to add a test.


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org