You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by tgravescs <gi...@git.apache.org> on 2018/06/12 14:27:49 UTC

[GitHub] spark pull request #21475: [SPARK-24416] Fix configuration specification for...

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

    https://github.com/apache/spark/pull/21475#discussion_r194759455
  
    --- Diff: docs/configuration.md ---
    @@ -1629,8 +1629,10 @@ Apart from these, the following properties are also available, and may be useful
       <td><code>spark.blacklist.killBlacklistedExecutors</code></td>
       <td>false</td>
       <td>
    -    (Experimental) If set to "true", allow Spark to automatically kill, and attempt to re-create,
    -    executors when they are blacklisted.  Note that, when an entire node is added to the blacklist,
    +    (Experimental) If set to "true", allow Spark to automatically kill the
    +    executors when they are blacklisted on fetch failure or blacklisted for the entire application, 
    +    as controlled by spark.blacklist.application.*.  
    +    Note that, when an entire node is added to the blacklist,
    --- End diff --
    
    very minor nit, can you put this on previous line and wrap text accordingly


---

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