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

    https://github.com/apache/spark/pull/22392#discussion_r217821786
  
    --- Diff: 
streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala ---
    @@ -54,6 +54,10 @@ class Checkpoint(ssc: StreamingContext, val 
checkpointTime: Time)
           "spark.driver.bindAddress",
           "spark.driver.port",
           "spark.master",
    +      "spark.kubernetes.driver.pod.name",
    +      "spark.kubernetes.driver.limit.cores",
    +      "spark.kubernetes.executor.limit.cores",
    --- End diff --
    
    Because, in my opinion, the previous checkpointed execution should not 
impact a newer choice of limit cores, nor should impose its old defaults on the 
newly-spawned executor pods.


---

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

Reply via email to