eolivelli commented on pull request #9396:
URL: https://github.com/apache/pulsar/pull/9396#issuecomment-771479204


   @codelipenghui I have checked `isBatchMessagingEnabled()`, and in my opinion 
changing that method will have a big impact and I will have to make changes in 
lots of points in ProducerImpl.
   I think it is not safe to change `isBatchMessagingEnabled()`
   
   I don't like that names for variables but I did not find a better name and 
also I wanted to keep the patch as small as possible


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

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


Reply via email to