You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by vanzin <gi...@git.apache.org> on 2018/06/28 23:00:58 UTC

[GitHub] spark pull request #21633: [SPARK-24646][CORE] Minor change to spark.yarn.di...

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

    https://github.com/apache/spark/pull/21633#discussion_r199011958
  
    --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala ---
    @@ -483,10 +483,11 @@ package object config {
     
       private[spark] val FORCE_DOWNLOAD_SCHEMES =
         ConfigBuilder("spark.yarn.dist.forceDownloadSchemes")
    -      .doc("Comma-separated list of schemes for which files will be downloaded to the " +
    +      .doc("Comma-separated list of schemes for which resources will be downloaded to the " +
             "local disk prior to being added to YARN's distributed cache. For use in cases " +
             "where the YARN service does not support schemes that are supported by Spark, like http, " +
    -        "https and ftp.")
    +        "https and ftp, or jars required to add to local YARN client's classpath. Wildcard '*' is" +
    --- End diff --
    
    "required to be in the local..."
    
    Also, keep the space at the end of this line, like done in others,


---

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