[GitHub] [arrow] danepitkin commented on issue #35370: Casting Timestamp scalar does not preserve UTC Suffix

["danepitkin (via GitHub)" <gi...@apache.org>]

danepitkin commented on issue #35370:
URL: https://github.com/apache/arrow/issues/35370#issuecomment-1530232971

   Good catch! I can reproduce the bug. The scalar code path does not handle timezone info. The pyarrow scalar cast uses a different implementation (`arrow/cpp/src/arrow/util/time.h::ConvertTimestampValue()`) than the pyarrow array cast (`arrow/compute/kernels/scalar_cast_string.cc::TemporalToStringCastFunctor`). Ideally, all functionality should be migrated to the compute kernel implementation. That will be a pretty big change, so the quickest option is to patch the current code path to add support for timezones.


-- 
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