[GitHub] [pinot] somandal commented on pull request #10570: [multistage] Split MailboxReceiveOperator into sorted and non-sorted versions

["somandal (via GitHub)" <gi...@apache.org>]

somandal commented on PR #10570:
URL: https://github.com/apache/pinot/pull/10570#issuecomment-1500522153

   > LGTM. However IIUC this only hides the issue of slow mailbox receiver (e.g. there still can be possibility that the seenMail signal got lost). did we capture this in any of the action items in #10424 ?
   > 
   > CC @ankitsultana please kindly share your feedback and we can follow up next
   
   I see @ankitsultana opened this PR: https://github.com/apache/pinot/pull/10564 to address one of the issues with the callback mechanisms. Will let @ankitsultana talk about any others that he alluded to in the issue: https://github.com/apache/pinot/issues/10555


-- 
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@pinot.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org