Re: [PR] [CALCITE-7494] Avatica conversion to string of TIMESTAMP WITH TIME ZONE does not include time zone [calcite-avatica]

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

xiedeyantu commented on PR #314:
URL: https://github.com/apache/calcite-avatica/pull/314#issuecomment-5563578122

   As I understand it, the issue is simply that Avatica was losing the "UTC" designation, so the goal of this PR (including the tests) is to verify that the fix preserves that designation. Therefore, there is no need to verify the correctness of the time zone handling itself (such as in the example below)—is that correct? 
   ```
   select TIMESTAMP WITH TIME ZONE '2020-01-01 00:00:00 America/New_York';
   +---------------------+
   | EXPR$0                    |
   +---------------------+
   | 2020-01-01 05:00:00 |
   +---------------------+
   ```


-- 
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: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org