advancedxy commented on PR #41181: URL: https://github.com/apache/spark/pull/41181#issuecomment-1550797386
> Thank you for making a PR, @turboFei . > > However, this PR might cause a outage because the number of configMap is controlled by quota. > > ``` > $ kubectl describe quota | grep configmaps > count/configmaps 4 150 > ``` > > To avoid the production outage, this should be under a new configuration with `false` by default at least. 150 is a bit small for serious production usage, we may add this note in the `running_on_k8s.md` documentation. And BTW, this MR doesn't create new ConfigMaps, it either uses a user pre-set config map (no creation) or just reuse the config map created by driver which is created if necessary. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
