| From: | Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>, pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org>, Alexander Korotkov <akorotkov(at)postgresql(dot)org>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pgsql: Implement jsonpath .datetime() method |
| Date: | 2019-10-14 02:36:39 |
| Message-ID: | CAPpHfdvtUeC1ed8bKpsuYj8Z9hkeLPnku3FyYPddGHgNZFGpwA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
On Sun, Oct 13, 2019 at 5:24 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> writes:
> > This patch also changes the way timestamp to timestamptz cast works.
> > Previously it did timestamp2tm() then tm2timestamp(). Instead, after
> > timestamp2tm() it calculates timezone offset and applies it to
> > original timestamp value. I hope this is correct.
>
> I'd wonder whether this gives the same answers near DST transitions,
> where it's not real clear which offset applies.
I will try this and share the results.
> Please *don't* wrap this sort of thing into an unrelated feature patch.
Sure, thank you for noticing.
------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tomas Vondra | 2019-10-14 21:40:02 | pgsql: Check for tables with sql_identifier during pg_upgrade |
| Previous Message | Michael Paquier | 2019-10-14 00:00:23 | Re: pgsql: Fix dependency handling of column drop with partitioned tables |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2019-10-14 06:02:27 | Re: Tighten error control for OpenTransientFile/CloseTransientFile |
| Previous Message | Amit Kapila | 2019-10-14 02:11:23 | Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions |