From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Add pg_alterckey utility to change the cluster key |
Date: | 2020-12-26 10:03:00 |
Message-ID: | alpine.DEB.2.22.394.2012260543130.2006135@pseudo |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Hello Bruce
Tom>> Possibly these commits need more review than you think.
Michaël> Shared feeling here, I think that this is still too early.
Michaël> FWIW, I am surprised that this patch series includes exactly zero line of code
Michaël> for tests, while the total amount of code committed is close to
Michaël> 3000 lines.
I intended to have a look at all that, but it got committed before I had
time to do it. Ok, I do not have much time these past months, but I've
been quite surprised anyway by the speed for such a feature/patch.
The feeling I expressed early in the thread is that the design should be
extendable, so that it does not fit only one particular use-case but fail
at any other that were not the author's, and a large reimplementation
would be needed to get them in. I was thinking of having a closer look
with that in mind. In particular and IMHO, the "master key" should not
(necessarily) be hold by a postgres process, not sure what the current
status is, but that is an example.
Committing such amount of codes and features without any test is much too
blunt. Why the rush?
Question: Should these patches be reverted till the stuff is properly
stabilized, tested and reviewed?
--
Fabien.
From | Date | Subject | |
---|---|---|---|
Next Message | Robins Tharakan | 2020-12-26 10:09:05 | Re: pgsql: Add scripts for retrieving the cluster file encryption key |
Previous Message | Michael Paquier | 2020-12-26 09:24:45 | Re: pgsql: Add pg_alterckey utility to change the cluster key |
From | Date | Subject | |
---|---|---|---|
Next Message | Dmitry Dolgov | 2020-12-26 10:46:35 | Re: pg_stat_statements and "IN" conditions |
Previous Message | Michael Paquier | 2020-12-26 09:24:45 | Re: pgsql: Add pg_alterckey utility to change the cluster key |