Github user maryannxue commented on a diff in the pull request:
https://github.com/apache/spark/pull/23249#discussion_r239693849
--- 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
--- End diff --
nit: add "," after "this manner".
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
