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 2025/06/02 07:06:38 UTC
[PR] fix(c/driver/postgresql): ingest zoned timestamp as WITH TIME ZONE [arrow-adbc]
lidavidm opened a new pull request, #2904:
URL: https://github.com/apache/arrow-adbc/pull/2904
Fixes #2901.
--
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
Re: [PR] fix(c/driver/postgresql): ingest zoned timestamp as WITH TIME ZONE [arrow-adbc]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot commented on PR #2904:
URL: https://github.com/apache/arrow-adbc/pull/2904#issuecomment-2935494323
Ah, sorry, I'd forgotten that timezone of "" makes it timezone unaware, not UTC. Ignore me!
--
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
Re: [PR] fix(c/driver/postgresql): ingest zoned timestamp as WITH TIME ZONE [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #2904:
URL: https://github.com/apache/arrow-adbc/pull/2904#issuecomment-2932888300
Ah, I will do some more testing...but I think zoned timestamps should be preserved as zoned timestamps (even if the actual metadata isn't preserved)
--
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
Re: [PR] fix(c/driver/postgresql): ingest zoned timestamp as WITH TIME ZONE [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm merged PR #2904:
URL: https://github.com/apache/arrow-adbc/pull/2904
--
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
Re: [PR] fix(c/driver/postgresql): ingest zoned timestamp as WITH TIME ZONE [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #2904:
URL: https://github.com/apache/arrow-adbc/pull/2904#issuecomment-2938732666
Added another test in Python, just to confirm that we don't need to reset the timezone before a bulk COPY (interestingly enough).
--
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
Re: [PR] fix(c/driver/postgresql): ingest zoned timestamp as WITH TIME ZONE [arrow-adbc]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot commented on PR #2904:
URL: https://github.com/apache/arrow-adbc/pull/2904#issuecomment-2933340769
Got it. No strong preference either way since the point in time is correct, just noting that we're substituting an arbitrary time zone in place of a type that we can at least predict.
--
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