sarutak commented on pull request #29002: URL: https://github.com/apache/spark/pull/29002#issuecomment-656247590
> Ideally, that way should work as well. However, it seems plugin initialization relies on some internal instances of Executor. So, if > we want to initialize plugin out of the executor, I'm afraid we need more changes. Yes, if we reorder the initialization with another way, the initialization would be put on executor backends like `CoarseGrainedExecutorBackend` and we might need more changes as you are afraid. > BTW, I think your current implementation can already work as I mentioned above. Yeah, I misunderstood. `LaunchExecutor` is sent after `Executor` is instansiated so the current implementation works. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
