juliuszsompolski commented on PR #42355: URL: https://github.com/apache/spark/pull/42355#issuecomment-1668673016
@hvanhovell > Is the ExecuteGrpcResponseSender thread safe? For example is detach() safe? detach() is synchronized on the executeObserver it's attached to, like the other places in the sender... Modified the code a bit to make it more clear. > Is it possible for events to produced in a weird order? No. Execution pushes responses to executeObserver in order, there is a bunch of assertions that assure order of events. -- 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]
