clownxc commented on code in PR #40638:
URL: https://github.com/apache/spark/pull/40638#discussion_r1164246953
##########
sql/catalyst/src/main/java/org/apache/spark/sql/connector/read/PartitionReaderFactory.java:
##########
@@ -55,7 +55,7 @@ default PartitionReader<ColumnarBatch>
createColumnarReader(InputPartition parti
}
/**
- * Returns true if the given {@link InputPartition} should be read by Spark
in a columnar way.
+ * @return true if the given {@link InputPartition} should be read by Spark
in a columnar way.
* This means, implementations must also implement {@link
#createColumnarReader(InputPartition)}
Review Comment:
> and here
Thank you very much for review and sorry for the late response. I will try
to modify the code according to your suggestions
--
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]
