cloud-fan commented on a change in pull request #35710:
URL: https://github.com/apache/spark/pull/35710#discussion_r827081236
##########
File path:
sql/catalyst/src/main/java/org/apache/spark/sql/connector/read/SupportsPushDownTopN.java
##########
@@ -31,8 +31,23 @@
@Evolving
public interface SupportsPushDownTopN extends ScanBuilder {
+ /**
+ * The first element of tuple means topN can be pushed down to data source.
+ * The second element of tuple is false means Spark complete push down
topN,
+ * otherwise means Spark partial push down topN and need preserves sort
too.
+ */
+ class Tuple {
+ public final Boolean enable;
+ public final Boolean isPartial;
Review comment:
2 boolean means we should have two methods... You did it right at the
beginning, why do you change the API?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
