Re: BUG #15954: Unable to alter partitioned table to set logged

From: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, "keith(dot)fiske(at)crunchydata(dot)com" <keith(dot)fiske(at)crunchydata(dot)com>, "ejberdecia(at)yahoo(dot)com" <ejberdecia(at)yahoo(dot)com>, "david(dot)rowley(at)2ndquadrant(dot)com" <david(dot)rowley(at)2ndquadrant(dot)com>, "amitlangote09(at)gmail(dot)com" <amitlangote09(at)gmail(dot)com>, "pgsql-bugs(at)lists(dot)postgresql(dot)org" <pgsql-bugs(at)lists(dot)postgresql(dot)org>
Subject: Re: BUG #15954: Unable to alter partitioned table to set logged
Date: 2024-04-23 12:33:00
Message-ID: CAKFQuwaEdwXv3=UsFUuNBvVg9r3MJzW-U-04E76ovi8LQFqvVQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Monday, April 22, 2024, Michael Paquier <michael(at)paquier(dot)xyz> wrote:

> On Mon, Apr 22, 2024 at 12:36:20PM -0700, David G. Johnston wrote:
> > There is little upside to preventing the existence of an unlogged
> > partitioned table at this point. I have just submitted a doc patch to
> deal
> > with that side of the equation.
>
> I would not object to that. However, now that we're five years into
> this problem, I'd rather treat that as a new entire feature than
> change the existing behavior in the back branches.
>
> Not changing unlogged property of a partitioned table on ALTER TABLE
> is of course no good.

>
Correct, it is an outright bug that I see no reason not to just fix and
back-patch. The new feature needs its own thread so as not to have this
one see the same fate it had 5 years ago.

David J.

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2024-04-23 14:39:53 Re: BUG #18445: date_part / extract range for hours do not match documentation
Previous Message Tender Wang 2024-04-23 11:14:04 Re: BUG #18377: Assert false in "partdesc->nparts >= pinfo->nparts", fileName="execPartition.c", lineNumber=1943