nicknezis commented on pull request #3710:
URL: https://github.com/apache/incubator-heron/pull/3710#issuecomment-926950066


   It seems that Spark has a [hardcoded 
key](https://github.com/apache/spark/blob/ff3f3c45668364da9bd10992791b5ee9a46fea21/resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Constants.scala#L86)
 for the ConfigMap, but I think this is because they create the ConfigMap and 
mount it. I like the flexibility of your suggested modification to the config 
value.
   
   I'm sorry that I've been a bit busy lately. I will try to review the code 
and test a bit tonight.
   
   Thanks again for helping add this feature. I'm already excited to add more 
subtle improvements to the Kubernetes scheduler. :)


-- 
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]


Reply via email to