yaooqinn commented on pull request #32830: URL: https://github.com/apache/spark/pull/32830#issuecomment-857473012
> You are right. Then, shall we move the check from `val driverPod` to `start` method? `ExecutorPodsAllocator` creation and `start` invocation is different? moved. the `val driverPod` itself seems also to have a regression, which is it may try to retrieve a non-existing driver pod for client mode if the `spark.kubernetes.driver.pod.name` is accidentally specified at the same time. Will check this later. -- 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]
