ARROW-7744: [Java][FlightRPC] JDBC Driver for Arrow Flight SQL #13800
Conversation
* Make UTC the default tz instead of using local tz * Use Avatica DateTimeUtils for getString in Date/Time/TimeStamp * Correct usage for unix*ToString * Add unit tests for Date/Time getString * Check wasNull
* Add support for getUnicodeStream * nit: remove outofscope changes * Fix small mistake in one of the added tests * nit: remove unused imports * Simplify PR
|
Great, CI looks good now (minus known extant issues). I'll give this a final review soon |
lidavidm
left a comment
lidavidm
left a comment
There was a problem hiding this comment.
As noted there are several items here to improve but I think we should get this in and keep working on it. I filed ARROW-17729 with several items to address and @avantgardnerio has some useful improvements that should be reviewed and merged too
|
CI failures are unrelated/addressed elsewhere |
|
Awesome news! Thank you very much for all the hard work that went into this. My company is very excited to make some PRs in the near future :) |
|
Benchmark runs are scheduled for baseline = 9c628b7 and contender = eb00620. eb00620 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
…e#13800) Lead-authored-by: James Duong <duong.james@gmail.com> Co-authored-by: iurysalino <iurysalino@gmail.com> Co-authored-by: Vinicius Fraga <62815192+vfraga@users.noreply.github.com> Signed-off-by: David Li <li.davidm96@gmail.com>
This PR bumps Apache Arrow version from 9.0.0 to 10.0.0. Main changes related to PyAmber: ## Java/Scala side: - JDBC Driver for Arrow Flight SQL ([13800](apache/arrow#13800)) - Initial implementation of immutable Table API ([14316](apache/arrow#14316)) - Substrait, transaction, cancellation for Flight SQL ([13492](apache/arrow#13492)) - Read Arrow IPC, CSV, and ORC files by NativeDatasetFactory ([13811](apache/arrow#13811), [13973](apache/arrow#13973), [14182](apache/arrow#14182)) - Add utility to bind Arrow data to JDBC parameters ([13589](apache/arrow#13589)) ## Python side: - The batch_readahead and fragment_readahead arguments for scanning Datasets are exposed in Python ([ARROW-17299](https://issues.apache.org/jira/browse/ARROW-17299)). - ExtensionArrays can now be created from a storage array through the pa.array(..) constructor ([ARROW-17834](https://issues.apache.org/jira/browse/ARROW-17834)). - Converting ListArrays containing ExtensionArray values to numpy or pandas works by falling back to the storage array ([ARROW-17813](https://issues.apache.org/jira/browse/ARROW-17813)). - Casting Tables to a new schema now honors the nullability flag in the target schema ([ARROW-16651](https://issues.apache.org/jira/browse/ARROW-16651)).
…e#13800) Lead-authored-by: James Duong <duong.james@gmail.com> Co-authored-by: iurysalino <iurysalino@gmail.com> Co-authored-by: Vinicius Fraga <62815192+vfraga@users.noreply.github.com> Signed-off-by: David Li <li.davidm96@gmail.com>
12 participants
No description provided.