From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, exclusion(at)gmail(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org |
Subject: | Re: BUG #16577: Segfault on altering a table located in a dropped tablespace |
Date: | 2020-10-28 13:59:24 |
Message-ID: | 1251838.1603893564@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> On 2020-Oct-28, Michael Paquier wrote:
>> Haven't thought of that approach, good idea! That would not be
>> backpatchable but that would be a solution that does not require
>> creating files where we don't need them. Did you begin to look at
>> that?
> I haven't started on this one yet, but I intend to do so shortly.
> Strictly speaking, we can still introduce a new category of pg_shdepend
> entries in back branches; it won't break anything that works today.
Yeah, as long as the patched version won't actively fail when those
pg_shdepend entries are missing, I don't think a backpatch is too
hazardous. It might be worth checking that the extra entries don't
create huge problems if one does downgrade after some of them exist
--- but my feeling for how that mechanism works is that it'd Just
Work, and indeed provide the missing DROP protection even without
explicit action by the backend.
I would not be too excited about offering instructions for people
to manually add/remove the dependency entries. The amount of
value added, versus the risks of bollixing things completely,
doesn't sound like a good tradeoff.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2020-10-28 14:01:16 | Re: posgres 12 bug (partitioned table) |
Previous Message | Alvaro Herrera | 2020-10-28 13:40:45 | Re: BUG #16577: Segfault on altering a table located in a dropped tablespace |