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

    https://github.com/apache/spark/pull/3107#discussion_r20060444
  
    --- Diff: docs/configuration.md ---
    @@ -563,8 +566,8 @@ Apart from these, the following properties are also 
available, and may be useful
         </ul>
       </td>
       <td>
    -    Default number of tasks to use across the cluster for distributed 
shuffle operations
    -    (<code>groupByKey</code>, <code>reduceByKey</code>, etc) when not set 
by user.
    +    Default number of output partitions for operations like 
<code>join</code>,
    --- End diff --
    
    Should this say "number of shuffle partitions" - it's slightly weird to me 
to say "output" when this refers to something that is totally internal to Spark 
- it's output on the map side but input on he read side. In other cases I think 
output tends to mean things like saving as HDFS data, etc.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to