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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
