| From: | Vik Fearing <vik(at)postgresfriends(dot)org> |
|---|---|
| To: | Surafel Temesgen <surafel3000(at)gmail(dot)com> |
| Cc: | Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com>, 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> |
| Subject: | Re: WIP: System Versioned Temporal Table |
| Date: | 2021-01-17 22:42:59 |
| Message-ID: | aae2d4a8-9d58-6e42-ef4b-b9da0f5fef91@postgresfriends.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 1/17/21 5:46 PM, Surafel Temesgen wrote:
> On Sat, Jan 16, 2021 at 10:12 PM Vik Fearing <vik(at)postgresfriends(dot)org>
> wrote:
>
>>
>> I haven't looked at this patch in a while, but I hope that ALTER TABLE t
>> ADD SYSTEM VERSIONING is not adding any columns. That is a bug if it does.
>>
>>
> Yes, that is how I implement it. I don't understand how it became a bug?
This is not good, and I see that DROP SYSTEM VERSIONING also removes
these columns which is even worse. Please read the standard that you
are trying to implement!
I will do a more thorough review of the functionalities in this patch
(not necessarily the code) this week.
--
Vik Fearing
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Heikki Linnakangas | 2021-01-17 22:52:12 | Re: Yet another fast GiST build |
| Previous Message | Peter Geoghegan | 2021-01-17 22:35:18 | Re: Yet another fast GiST build |