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 2019/03/18 09:40:54 UTC

[GitHub] [pulsar] se7enkings opened a new pull request #3852: [Issue-3850]Fix the loop of wrapper consumer poll

se7enkings opened a new pull request #3852: [Issue-3850]Fix the loop of wrapper consumer poll
URL: https://github.com/apache/pulsar/pull/3852
 
 
   Fixes [3850](https://github.com/apache/pulsar/issues/3850)
   
   *Motivation*
   
   When we call the function named "poll(timeoutMillis)" to get records, the conditional judgment `++numberOfRecords < MAX_RECORDS_IN_SINGLE_POLL` is always true from beginning, which will  break the loop right now. So we only can get one record from the `poll()` each time.
   
   *Modifications*
   
   Change the conditional judgment to `++numberOfRecords >= MAX_RECORDS_IN_SINGLE_POLL`,
   which can cache more than one record in a signal poll.

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


With regards,
Apache Git Services