advancedxy commented on PR #41257: URL: https://github.com/apache/spark/pull/41257#issuecomment-1581928535
> We need to keep the existing behavior because this is designed to get a config-free executor pod. I'd like some clarifications before going further: 1. `config-free` executor pod, do you mean a `ConfigMap`-free executor pod or an executor pod without SPARK_CONF_DIR mounted? This pr wouldn't create a new config map for executor, so this is kind of ConfigMap free for executors. 2. by saying existing behavior, do you mean the `disableConfigMap` setting to control whether to create a new config map or not, or rather the old behavior to create ConfigMaps for executors? -- 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]
