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

    https://github.com/apache/spark/pull/20657#discussion_r172579936
  
    --- Diff: 
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
    @@ -520,4 +520,16 @@ package object config {
           .checkValue(v => v > 0, "The threshold should be positive.")
           .createWithDefault(10000000)
     
    +  private[spark] val CREDENTIALS_RENEWAL_INTERVAL_RATIO =
    +    ConfigBuilder("spark.security.credentials.renewalRatio")
    +      .doc("Ratio of the credential's expiration time when Spark should 
fetch new credentials.")
    +      .doubleConf
    +      .createWithDefault(0.75d)
    +
    +  private[spark] val CREDENTIALS_RENEWAL_RETRY_WAIT =
    +    ConfigBuilder("spark.security.credentials.retryWait")
    +      .doc("How long to wait before retrying to fetch new credentials 
after a failure.")
    +      .timeConf(TimeUnit.SECONDS)
    +      .createWithDefaultString("1h")
    --- End diff --
    
    I thought that is what internal meant ... a user *could* specify them, but 
we don't document them at all, so not a stable part of the api etc.


---

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

Reply via email to