ankitsultana commented on issue #10140: URL: https://github.com/apache/pinot/issues/10140#issuecomment-1396973491
Side-note: I think entries in seenMail may get leaked if `_releaseTs` is set to a finite value. The reason is that the OpChain may hang up and the sender may send the data after the corresponding OpChain has already finished as far as the scheduler is considered. It's only a small leak but a leak nevertheless. Broadly speaking I think we need to gracefully handle the cases where a sender is ready to send data to a stage but the receiver either hasn't been registered yet or has already hung up. > Separately, we should also see if we can bound the GRPC worker threads +1 -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
