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

    https://github.com/apache/spark/pull/23249#discussion_r239694008
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala
 ---
    @@ -243,10 +248,19 @@ case class HashPartitioning(expressions: 
Seq[Expression], numPartitions: Int)
      * Represents a partitioning where rows are split across partitions based 
on some total ordering of
      * the expressions specified in `ordering`.  When data is partitioned in 
this manner the following
      * two conditions are guaranteed to hold:
    - *  - All row where the expressions in `ordering` evaluate to the same 
values will be in the same
    --- End diff --
    
    nit: "row" -> "rows", "where... `ordering`" -> "whose `ordering` 
expressions"


---

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

Reply via email to