srowen commented on a change in pull request #25856: [SPARK-29182][Core] Cache
preferred locations of checkpointed RDD
URL: https://github.com/apache/spark/pull/25856#discussion_r329183879
##########
File path: core/src/main/scala/org/apache/spark/internal/config/package.scala
##########
@@ -783,6 +783,24 @@ package object config {
.booleanConf
.createWithDefault(false)
+ private[spark] val CACHE_CHECKPOINT_PREFERRED_LOCS =
+ ConfigBuilder("spark.rdd.checkpoint.cachePreferredLocs")
+ .internal()
+ .doc("Whether to cache preferred locations of checkpointed RDD. Caching
preferred " +
+ "locations can relieve query loading to DFS and save the query time.
The drawback " +
+ "is that the cached locations can be possibly outdated and lose data
locality. " +
+ "This location cache expires by
`spark.rdd.checkpoint.cachePreferredLocsExpireTime` " +
+ "minutes.")
+ .booleanConf
+ .createWithDefault(false)
+
+ private[spark] val CACHE_CHECKPOINT_PREFERRED_LOCS_EXPIRE_TIME =
Review comment:
Can we have one optional config instead of two? like, if the time isn't
specified or is 0, don't cache?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
