From: | Surafel Temesgen <surafel3000(at)gmail(dot)com> |
---|---|
To: | Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com> |
Cc: | Ryan Lambert <ryan(at)rustprooflabs(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Rémi Lapeyre <remi(dot)lapeyre(at)lenstra(dot)fr>, Eli Marmor <eli(at)netmask(dot)it>, David Steele <david(at)pgmasters(dot)net>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Georgios <gkokolatos(at)protonmail(dot)com>, Vik Fearing <vik(dot)fearing(at)enterprisedb(dot)com> |
Subject: | Re: WIP: System Versioned Temporal Table |
Date: | 2021-01-14 17:42:21 |
Message-ID: | CALAY4q-UhYNK-RaiNQ7ZzU64-CYHBXYXvMrqe6vumFaF0o4Yew@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Simon,
Thank you for all the work you does
On Mon, Jan 11, 2021 at 5:02 PM Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com>
wrote:
>
>
> * Anomalies around use of CURRENT_TIMESTAMP are not discussed or resolved.
> Probably need to add a test that end_timestamp > start_timestamp or ERROR,
> which effectively enforces serializability.
>
>
This scenario doesn't happen. There are no possibility of a record being
deleted or updated before inserting
> * No discussion, comments or tests around freezing and whether that
> causes issues here
>
>
This feature introduced no new issue regarding freezing. Adding
the doc about the table size growth because of a retention of old record
seems
enough for me
>
> * ALTER TABLE needs some work, it's a bit klugey at the moment and
> needs extra tests.
> Should refuse DROP COLUMN on a system time column, but currently doesn't
>
> * UPDATE foo SET start_timestamp = DEFAULT should fail but currently
> doesn't
>
>
okay i will fix it
regards
Surafel
From | Date | Subject | |
---|---|---|---|
Next Message | Surafel Temesgen | 2021-01-14 17:46:41 | Re: WIP: System Versioned Temporal Table |
Previous Message | Magnus Hagander | 2021-01-14 17:29:28 | Re: vacuum_cost_page_miss default value and modern hardware |