You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by srowen <gi...@git.apache.org> on 2018/08/01 15:40:50 UTC

[GitHub] spark pull request #21825: [SPARK-18188][DOC][FOLLOW-UP]Add `spark.broadcast...

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

    https://github.com/apache/spark/pull/21825#discussion_r206931690
  
    --- Diff: docs/configuration.md ---
    @@ -1215,6 +1215,15 @@ Apart from these, the following properties are also available, and may be useful
         if it is too small, <code>BlockManager</code> might take a performance hit.
       </td>
     </tr>
    +<tr>
    +  <td><code>spark.broadcast.checksum</code></td>
    +  <td>true</td>
    +  <td>
    +    Whether to enable checksum for broadcast.If enabled (default), broadcasts will include a checksum, which
    --- End diff --
    
    Final nit: add the space after the period.


---

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