peter-toth commented on PR #52899: URL: https://github.com/apache/spark/pull/52899#issuecomment-3495480310
> We’ve made similar changes in our production environment, so this part of the code > > https://github.com/apache/spark/blob/562f683fa4e2dc8bdb9e2f157ece8c6544580f55/resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsLifecycleManager.scala#L212 > > > shouldn’t be needed anymore, right? The delete calls are very expensive so if we can avoid it with a cheap get we should do it. BTW this part is further improved in https://github.com/apache/spark/pull/52902 and explaned in the PR description why we should skip delete if not needed. -- 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]
