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-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to