Github user tgravescs commented on the pull request:

    https://github.com/apache/spark/pull/3409#issuecomment-66343795
  
    the spark.yarn.applicationMaster.waitTries is being changed in a pull 
request to change it from tries to time and there is another one to add memory 
overhead param.  I was trying to make all 3 of those use the same convention.  
So lets decide on it here and we can enforce in those.  The only one not 
covered is the spark.yarn.appMasterEnv 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to