Github user witgo commented on the pull request:

    https://github.com/apache/spark/pull/969#issuecomment-46198873
  
    @tgravescs  The code should be like this?
    
    ClientArguments:
    ```scala
      // -archives/--files via spark submit or yarn-client defaults to use 
file:// if not specified
      if (sys.props.contains("SPARK_SUBMIT") || 
(sparkConf.getOption("spark.master").isDefined &&
        sparkConf.get("spark.master") == "yarn-client")) {
        files = Option(files).map(p => Utils.resolveURIs(p)).orNull
        archives = Option(archives).map(p => Utils.resolveURIs(p)).orNull
      }
    
      // env variable SPARK_YARN_DIST_ARCHIVES/SPARK_YARN_DIST_FILES set in 
yarn-client then
      // it should default to hdfs://
      files = 
Option(files).getOrElse(sys.props.get("SPARK_YARN_DIST_FILES").orNull)
      archives = 
Option(archives).getOrElse(sys.props.get("SPARK_YARN_DIST_ARCHIVES").orNull)
    
      // spark.yarn.dist.archives/spark.yarn.dist.files defaults to use file:// 
if not specified,
      // for both yarn-client and yarn-cluster
      files = 
Option(files).getOrElse(sparkConf.getOption("spark.yarn.dist.files").
        map(p => Utils.resolveURIs(p)).orNull)
      archives = 
Option(archives).getOrElse(sparkConf.getOption("spark.yarn.dist.archives").
        map(p => Utils.resolveURIs(p)).orNull)
    
    ```


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to