lidavidm commented on code in PR #14168:
URL: https://github.com/apache/arrow/pull/14168#discussion_r974444466
##########
java/dataset/src/main/java/org/apache/arrow/dataset/jni/NativeScanner.java:
##########
@@ -68,6 +68,15 @@ ArrowReader execute() {
}
@Override
+ public NativeScanTask scanTask() {
Review Comment:
The deprecation makes sense to me, I think we should consider what the new
interface is, though. I would think something like `ArrowReader
scanBatchesUnordered()`.
##########
java/dataset/src/main/java/org/apache/arrow/dataset/jni/NativeScanner.java:
##########
@@ -68,6 +68,15 @@ ArrowReader execute() {
}
@Override
+ public NativeScanTask scanTask() {
Review Comment:
That would mirror the C++ API
--
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]
