-
Notifications
You must be signed in to change notification settings - Fork 4k
MINOR: [Docs][Java] Update status documentation for Java's CSV read support (as per ARROW-17786) #34390
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Update status documentation: CSV read in Java from #14182
|
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format? or In the case of PARQUET issues on JIRA the title also supports: See also: |
lidavidm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, just one comment.
docs/source/status.rst
Outdated
| | Avro | | R | | | | | | | ||
| +-----------------------------+---------+---------+-------+------------+-------+-------+-------+ | ||
| | CSV | R/W | | R/W | | | R/W | R/W | | ||
| | CSV | R/W | R | R/W | | | R/W | R/W | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add the caveat that it's through JNI (like for ORC below).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @lidavidm Thank you for review! Done
lidavidm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
|
Benchmark runs are scheduled for baseline = 2ebb3f7 and contender = 5a7d89c. 5a7d89c is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
|
['Python', 'R'] benchmarks have high level of regressions. |
Update status documentation: CSV read in Java from #14182