Github user liyinan926 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22146#discussion_r212068833
  
    --- Diff: 
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Constants.scala
 ---
    @@ -74,8 +74,15 @@ private[spark] object Constants {
       val ENV_R_PRIMARY = "R_PRIMARY"
       val ENV_R_ARGS = "R_APP_ARGS"
     
    +  // Pod spec templates
    +  val EXECUTOR_POD_SPEC_TEMPLATE_FILE_NAME = "podSpecTemplate.yml"
    --- End diff --
    
    Can we not use camel case for the file name? This looks inconsistent with 
other conf files. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to