On Jul 22, 2024, at 03:12, Jeevan Chalke <jeevan.cha...@enterprisedb.com> wrote:

> I agree with David that we need to set the tz explicitly as the JsonbValue 
> struct maintains that separately.
> 
> However, in the attached version, I have added some comments and also, fixed 
> some indentation.

Thank you for the review. I changed a single word in your comments (which are 
welcome). Thank you!

Just to reiterate, this is not an ideal fix, as the `date_timestamptz` and 
`timestamp_timestamptz` perform the same calculations. It would be nice to do 
it only once.

Best,

David

Attachment: v5-0001-Preserve-tz-when-converting-to-jsonb-timestamptz.patch
Description: Binary data


Reply via email to