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

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

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

   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 https://github.com/apache/pinot/issues/10424 ?
   
   CC @ankitsultana please kindly share your feedback and we can follow up next


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