vibhatha commented on code in PR #36967:
URL: https://github.com/apache/arrow/pull/36967#discussion_r1371046191
##########
java/dataset/src/main/java/org/apache/arrow/dataset/jni/JniWrapper.java:
##########
@@ -76,6 +76,22 @@ private JniWrapper() {
*/
public native long createScanner(long datasetId, String[] columns, long
batchSize, long memoryPool);
+ /**
+ * Create Scanner from a Dataset and get the native pointer of the Dataset.
+ * @param datasetId the native pointer of the arrow::dataset::Dataset
instance.
+ * @param columns desired column names.
+ * Columns not in this list will not be emitted when
performing scan operation. Null equals
+ * to "all columns".
+ * @param filter the filter to apply on the scan
Review Comment:
nit: usage of period at the end of a comment. Certain parameters do have it
but some parameters don't.
--
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]
