Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21145#discussion_r186587220
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/ReadTask.java ---
@@ -22,20 +22,20 @@
import org.apache.spark.annotation.InterfaceStability;
/**
- * A reader factory returned by {@link
DataSourceReader#createDataReaderFactories()} and is
+ * A read task returned by {@link DataSourceReader#createReadTasks()} and
is
--- End diff --
Okay, one more idea that captures the relationship between `DataReader` and
`ReadTask`: What about using `DataReadable`? That's similar to the `Iterable`
and `Iterator` relationship that the two classes have.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
