cloud-fan commented on a change in pull request #34073:
URL: https://github.com/apache/spark/pull/34073#discussion_r714445044
##########
File path:
sql/catalyst/src/main/java/org/apache/spark/sql/connector/read/SupportsPushDownV2Filters.java
##########
@@ -22,23 +22,26 @@
/**
* A mix-in interface for {@link ScanBuilder}. Data sources can implement this
interface to
- * push down filters to the data source and reduce the size of the data to be
read.
Review comment:
Let's only change the classdoc
```
push down V2 {@link Filter}s to ...
Note that, this interface is preferred over {@link SupportsPushDownFilters},
which uses V1 Filter and is less
efficient due to the internal -> external data conversion.
```
--
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]
