sunchao commented on a change in pull request #31355:
URL: https://github.com/apache/spark/pull/31355#discussion_r565769723



##########
File path: 
sql/catalyst/src/main/java/org/apache/spark/sql/connector/distributions/OrderedDistribution.java
##########
@@ -32,4 +32,13 @@
    * Returns ordering expressions.
    */
   SortOrder[] ordering();
+
+  /**
+   * Returns the number of partitions required by this write.

Review comment:
       I'm not sure if `required by this write` is accurate since we could 
potentially reuse this interface for read as well in future?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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

Reply via email to