[GitHub] [pulsar] rdhabalia commented on pull request #7266: [pulsar-broker] Dispatch batch messages according consumer permits

[GitBox <gi...@apache.org>]

rdhabalia commented on pull request #7266:
URL: https://github.com/apache/pulsar/pull/7266#issuecomment-643660448


   @codelipenghui 
   I saw  #6719 but it doesn't solve fundamental problem of dispatching.  #6719 tries to **predict** how many messages should be read by considering total-available permits and total batch messages. but it's still not controlling while dispatching messages. 
   for example: there are 10 consumers connected with 100 permits each and one batch message contains 100 messages. Now, broker reads 10 messages based on both factors but broker doesn't know number of batch-messages in each message so, it will dispatch all 10 messages to the consumer which has 100 permits which is the actual issue because broker dispatched 10*100=1K message to the consumer. 


----------------------------------------------------------------
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:
users@infra.apache.org