Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21326#discussion_r189730557
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/DataSourceReader.java
---
@@ -59,21 +59,21 @@
* Returns the actual schema of this data source reader, which may be
different from the physical
* schema of the underlying storage, as column pruning or other
optimizations may happen.
*
- * If this method fails (by throwing an exception), the action would
fail and no Spark job was
+ * If this method fails (by throwing an exception), the action will fail
and no Spark job will be
* submitted.
*/
StructType readSchema();
/**
- * Returns a list of read tasks. Each task is responsible for creating a
data reader to
- * output data for one RDD partition. That means the number of tasks
returned here is same as
- * the number of RDD partitions this scan outputs.
+ * Returns a list of {@link InputPartition}s. Each {@link
InputPartition} is responsible for
--- End diff --
Instead of `{@link InputPartition}s` use `{@link InputPartition
partitions}` to add the plural. That avoids awkward formatting and links that
are missing 's'.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
