sunchao commented on a change in pull request #31355:
URL: https://github.com/apache/spark/pull/31355#discussion_r565840043
##########
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:
Agree this is not quite useful for read. Does it make sense to put this
in `RequiresDistributionAndOrdering`? it will deviate from the catalyst
definition of distribution but this is for write path only.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
