You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by gaborgsomogyi <gi...@git.apache.org> on 2017/11/27 13:41:15 UTC

[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

GitHub user gaborgsomogyi opened a pull request:

    https://github.com/apache/spark/pull/19826

    [SPARK-22428][DOC] Add spark application garbage collector configurat…

    ## What changes were proposed in this pull request?
    
    The spark properties for configuring the ContextCleaner are not documented in the official documentation at https://spark.apache.org/docs/latest/configuration.html#available-properties.
    
    This PR adds the doc.
    
    ## How was this patch tested?
    
    Manual.
    
    ```
    cd docs
    jekyll build
    open _site/configuration.html
    ```


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gaborgsomogyi/spark SPARK-22428

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/19826.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #19826
    
----
commit cbd9d688d13a86c45ef57ae81a3b0b3b05fa65c1
Author: gaborgsomogyi <ga...@gmail.com>
Date:   2017-11-27T13:32:52Z

    [SPARK-22428][DOC] Add spark application garbage collector configuration doc

----


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by gaborgsomogyi <gi...@git.apache.org>.
Github user gaborgsomogyi commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153832461
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,51 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    --- End diff --
    
    Moved as asked.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #3995 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3995/testReport)** for PR 19826 at commit [`bd6c2bb`](https://github.com/apache/spark/commit/bd6c2bb6de98c37095a824fb9dac0d6e9afecf57).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by gaborgsomogyi <gi...@git.apache.org>.
Github user gaborgsomogyi commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153430954
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    --- End diff --
    
    Given more detailed description.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    Can one of the admins verify this patch?


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by gaborgsomogyi <gi...@git.apache.org>.
Github user gaborgsomogyi commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153431216
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether a ContextCleaner should be created when a SparkContext initializes.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking.blocking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether the cleaning thread should block on cleanup tasks (other than shuffle, which is controlled by
    +      spark.cleaner.referenceTracking.blocking.shuffle Spark property).<br><br>
    +      It is true as a workaround to <a href="https://issues.apache.org/jira/browse/SPARK-3015">
    --- End diff --
    
    Removed.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #3994 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3994/testReport)** for PR 19826 at commit [`cbd9d68`](https://github.com/apache/spark/commit/cbd9d688d13a86c45ef57ae81a3b0b3b05fa65c1).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r154134827
  
    --- Diff: docs/configuration.md ---
    @@ -2306,7 +2346,6 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    -
    --- End diff --
    
    Maybe I am insane but let's revert unneeded change


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by srowen <gi...@git.apache.org>.
Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    Merged to master


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84358/
    Test PASSed.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    retest this please


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #84343 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84343/testReport)** for PR 19826 at commit [`d117c9f`](https://github.com/apache/spark/commit/d117c9f0b626e3041066faf5d22d0f75f1884025).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by gaborgsomogyi <gi...@git.apache.org>.
Github user gaborgsomogyi commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153431150
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether a ContextCleaner should be created when a SparkContext initializes.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking.blocking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether the cleaning thread should block on cleanup tasks (other than shuffle, which is controlled by
    +      spark.cleaner.referenceTracking.blocking.shuffle Spark property).<br><br>
    --- End diff --
    
    Surrounded.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by srowen <gi...@git.apache.org>.
Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153211458
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether a ContextCleaner should be created when a SparkContext initializes.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking.blocking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether the cleaning thread should block on cleanup tasks (other than shuffle, which is controlled by
    +      spark.cleaner.referenceTracking.blocking.shuffle Spark property).<br><br>
    +      It is true as a workaround to <a href="https://issues.apache.org/jira/browse/SPARK-3015">
    --- End diff --
    
    I think you can remove comments like this, or leave them to the source code. This doesn't directly help the reader understand when or if to change it. Or, add context here about why one would set this.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by gaborgsomogyi <gi...@git.apache.org>.
Github user gaborgsomogyi commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153431033
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether a ContextCleaner should be created when a SparkContext initializes.
    --- End diff --
    
    Rephrased.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #84358 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84358/testReport)** for PR 19826 at commit [`241dbd0`](https://github.com/apache/spark/commit/241dbd069929c9f9eb6f1f9e45485693c9d4cd60).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by srowen <gi...@git.apache.org>.
Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153211780
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    --- End diff --
    
    You might give at least a sentence of explanation about what this has to do with the context cleaner.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    Merged build finished. Test PASSed.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by srowen <gi...@git.apache.org>.
Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153827867
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,51 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    --- End diff --
    
    Pardon, I just noticed one more thing. This is really about the Context Cleaner, right? not garbage collection per se. I'd probably put these properties under Memory Management instead of a new section.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    Merged build finished. Test PASSed.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by gaborgsomogyi <gi...@git.apache.org>.
Github user gaborgsomogyi commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r154156746
  
    --- Diff: docs/configuration.md ---
    @@ -2306,7 +2346,6 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    -
    --- End diff --
    
    Reverted.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    retest this please


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84343/
    Test PASSed.


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by srowen <gi...@git.apache.org>.
Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153212103
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether a ContextCleaner should be created when a SparkContext initializes.
    --- End diff --
    
    I'd just say it controls whether context cleaning is enabled at all. This sort of suggests it controls whether it's created at a certain time as opposed to another time.


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #84358 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84358/testReport)** for PR 19826 at commit [`241dbd0`](https://github.com/apache/spark/commit/241dbd069929c9f9eb6f1f9e45485693c9d4cd60).


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/spark/pull/19826


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #3994 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3994/testReport)** for PR 19826 at commit [`cbd9d68`](https://github.com/apache/spark/commit/cbd9d688d13a86c45ef57ae81a3b0b3b05fa65c1).


---

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


[GitHub] spark pull request #19826: [SPARK-22428][DOC] Add spark application garbage ...

Posted by srowen <gi...@git.apache.org>.
Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19826#discussion_r153211514
  
    --- Diff: docs/configuration.md ---
    @@ -2306,6 +2306,50 @@ showDF(properties, numRows = 200, truncate = FALSE)
       </tr>
     </table>
     
    +### Spark Application Garbage Collector
    +
    +<table class="table">
    +  <tr><th>Property Name</th><th>Default</th><th>Meaning</th></tr>
    +  <tr>
    +    <td><code>spark.cleaner.periodicGC.interval</code></td>
    +    <td>30min</td>
    +    <td>
    +      Controls how often to trigger a garbage collection.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether a ContextCleaner should be created when a SparkContext initializes.
    +    </td>
    +  </tr>
    +  <tr>
    +    <td><code>spark.cleaner.referenceTracking.blocking</code></td>
    +    <td>true</td>
    +    <td>
    +      Controls whether the cleaning thread should block on cleanup tasks (other than shuffle, which is controlled by
    +      spark.cleaner.referenceTracking.blocking.shuffle Spark property).<br><br>
    --- End diff --
    
    You might surround prop names in `<code>` for extra clarity


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #84343 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84343/testReport)** for PR 19826 at commit [`d117c9f`](https://github.com/apache/spark/commit/d117c9f0b626e3041066faf5d22d0f75f1884025).


---

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


[GitHub] spark issue #19826: [SPARK-22428][DOC] Add spark application garbage collect...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/19826
  
    **[Test build #3995 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3995/testReport)** for PR 19826 at commit [`bd6c2bb`](https://github.com/apache/spark/commit/bd6c2bb6de98c37095a824fb9dac0d6e9afecf57).


---

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