From: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
To: michael@paquier.xyz
Cc: keith.fiske@crunchydata.com
Cc: ejberdecia@yahoo.com
Cc: david.rowley@2ndquadrant.com
Cc: amitlangote09@gmail.com
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #15954: Unable to alter partitioned table to set logged
Date: Thu, 22 Aug 2019 17:14:39 +0900 (Tokyo Standard Time)
Message-ID: <20190822.171439.14634665.horikyota.ntt@gmail.com> (raw)
In-Reply-To: <20190821134434.GA9383@paquier.xyz>
References: <1703025078.5205270.1565786782053@mail.yahoo.com>
<CAODZiv58QvbbHCfh9MbbM6Bfuj7uSNjdCpt8r_H4VpmAOZ_9Qw@mail.gmail.com>
<20190821134434.GA9383@paquier.xyz>
At Wed, 21 Aug 2019 22:44:34 +0900, Michael Paquier <michael@paquier.xyz> wrote in <20190821134434.GA9383@paquier.xyz>
> On Wed, Aug 21, 2019 at 09:24:38AM -0400, Keith Fiske wrote:
> > As the others have stated, I think we just need to get some sort of
> > consistent method of handling this. Currently running an ALTER TABLE on the
> > parent to change the UNLOGGED state simply does nothing which is not
> > intuitive whatsoever. Even if it's just throwing an error saying you cannot
> > change this property, that would be better until a more thorough solution
> > can be implemented in the future.
>
> One problem with an error is that it may break existing application
> code :(
It is quite strange that we can CREATE both LOGGED and UNLOGGED
partitioned table but cannot ALTER the property. I believe no one
does ALTER TABLE SET (UN)LOGGED expecting it is silently ignored.
But I'm not sure about the CREATE case.. CREATE UNLOGGED VIEW is
complained as "cannot be unlogged bacause they do not have
storage" but I don't think partitioned tables don't necessarily
need to behave so. Even if any, I'm not sure what those who does
"CREATE UNLOGGED TABLE parent" expect (*), I think we can ignore
and always make the new table as LOGGED. The only trouble case is
happen only for those who checks the property.
*: Maybe they expect that the property propagates to children,
but anyway we are not doing so.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-bugs@postgresql.org
Cc: horikyota.ntt@gmail.com, michael@paquier.xyz, keith.fiske@crunchydata.com, ejberdecia@yahoo.com, david.rowley@2ndquadrant.com, amitlangote09@gmail.com, pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #15954: Unable to alter partitioned table to set logged
In-Reply-To: <20190822.171439.14634665.horikyota.ntt@gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox