You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "lidavidm (via GitHub)" <gi...@apache.org> on 2023/05/31 22:18:45 UTC
[GitHub] [arrow-adbc] lidavidm opened a new pull request, #722: test(java/driver/jdbc): check type system mapping
lidavidm opened a new pull request, #722:
URL: https://github.com/apache/arrow-adbc/pull/722
Fixes #720.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-adbc] lidavidm commented on pull request #722: feat(java/driver/jdbc): add hooks for JDBC type system mapping
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #722:
URL: https://github.com/apache/arrow-adbc/pull/722#issuecomment-1577266608
We may need custom column converters per database; it seems Postgres returns millis for timestamps (even if the value is in micros), though there may be a way to get the value still (https://github.com/pgjdbc/pgjdbc/issues/1537). But that also needs to be resolved upstream.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-adbc] lidavidm commented on pull request #722: feat(java/driver/jdbc): add hooks for JDBC type system mapping
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #722:
URL: https://github.com/apache/arrow-adbc/pull/722#issuecomment-1579370055
I'm going to merge this as a first start, but I think what we'll need is a `JdbcQuirks` that encapsulates details like this (type conversions, but also a SqlQuirks and eventually column converters) for each database.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-adbc] lidavidm merged pull request #722: feat(java/driver/jdbc): add hooks for JDBC type system mapping
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm merged PR #722:
URL: https://github.com/apache/arrow-adbc/pull/722
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org