linlinnn edited a comment on pull request #10240: URL: https://github.com/apache/pulsar/pull/10240#issuecomment-826029000
@MarvinCai Sorry, I do not get your point. In the scene of the last message(there is no more message come to invoke notifyPendingReceive) IMO, this is a trade-off concerns, if we add lock, we will lose 45% performance, I prefer to add a monitor thread to deal with the boundary case even though makeing the receiving of messages less timely. -- 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]
