From: Tom Lane <tgl@sss.pgh.pa.us>
To: Michael Paquier <michael@paquier.xyz>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Set pg_class.relhassubclass for partitioned indexes
Date: Sun, 21 Oct 2018 23:20:04 -0400
Message-ID: <82549.1540178404@sss.pgh.pa.us> (raw)
In-Reply-To: <20181022031212.GF14282@paquier.xyz>
References: <E1gEPd3-0007fj-6U@gemulon.postgresql.org>
<82470.1540177167@sss.pgh.pa.us>
<20181022031212.GF14282@paquier.xyz>
Michael Paquier <michael@paquier.xyz> writes:
> On Sun, Oct 21, 2018 at 10:59:27PM -0400, Tom Lane wrote:
>> Seems like this commit should have touched the catalogs.sgml description
>> for that column, as well as the pg_class.h comment for it. Neither of
>> those are worded in a way that suggests it could be set for non-table
>> relations.
> The pg_class.h comment looked fine for me first. How would you reword
> it?
Well, the question is what "derived class" means, but I'd tend to think
it means something that has an associated composite type; which indexes
do not. So maybe instead of "has (or has had) derived classes", we could
write "has (or has had) child tables or indexes"? I'm not wedded to
particular wording for this, but I think what's there now is a bit
misleading.
> relispartition tells "True if table is a partition", which is not
> actually true as it can apply to indexes. So this should be changed in
> v11 as well, no?
Good point, that column's description is obsolete as well.
regards, tom lane
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-committers@postgresql.org
Cc: tgl@sss.pgh.pa.us, michael@paquier.xyz, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Set pg_class.relhassubclass for partitioned indexes
In-Reply-To: <82549.1540178404@sss.pgh.pa.us>
* 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