HeartSaVioR commented on a change in pull request #31355: URL: https://github.com/apache/spark/pull/31355#discussion_r565860036
########## 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: That was my first trial, but stepped back because the number is only used for specific distributions. Probably I was over-thinking. Let me give it a try again. ---------------------------------------------------------------- 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