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