Doc patch: mention indexes in pg_inherits docs

Started by Dagfinn Ilmari Mannsåkerover 5 years ago4 messageshackers
Jump to latest

Hi Hackers,

When partitioned index support was added in veresion 11, the pg_inherits
docs missed the memo and still only say it describes table inheritance.
The attached patch adds mentions of indexes too, and notes that they can
not participate in multiple inheritance.

I don't know what the policy is on backpatching doc fixes, but
personally I think it should be.

- ilmari
--
"A disappointingly low fraction of the human race is,
at any given time, on fire." - Stig Sandbeck Mathisen

Attachments:

0001-Mention-that-pg_inherit-can-also-reference-indexes.patchtext/x-diffDownload+9-6
#2Michael Paquier
michael@paquier.xyz
In reply to: Dagfinn Ilmari Mannsåker (#1)
Re: Doc patch: mention indexes in pg_inherits docs

On Tue, Jul 28, 2020 at 12:21:29PM +0100, Dagfinn Ilmari Mannsåker wrote:

When partitioned index support was added in veresion 11, the pg_inherits
docs missed the memo and still only say it describes table inheritance.
The attached patch adds mentions of indexes too, and notes that they can
not participate in multiple inheritance.

What you have here looks fine to me. We could be more picky regarding
the types or relations that can be added, as it can actually be
possible to have a partitioned table or index, two relkinds of their
own, but what you are proposing looks fine enough here.

I don't know what the policy is on backpatching doc fixes, but
personally I think it should be.

This is actually a bug fix, because we include in the docs some
incorrect information, so it should be backpatched. If there are no
objections, I'll take care of that.
--
Michael

#3Michael Paquier
michael@paquier.xyz
In reply to: Michael Paquier (#2)
Re: Doc patch: mention indexes in pg_inherits docs

On Wed, Jul 29, 2020 at 03:06:58PM +0900, Michael Paquier wrote:

This is actually a bug fix, because we include in the docs some
incorrect information, so it should be backpatched. If there are no
objections, I'll take care of that.

And done.
--
Michael

In reply to: Michael Paquier (#3)
Re: Doc patch: mention indexes in pg_inherits docs

Michael Paquier <michael@paquier.xyz> writes:

On Wed, Jul 29, 2020 at 03:06:58PM +0900, Michael Paquier wrote:

This is actually a bug fix, because we include in the docs some
incorrect information, so it should be backpatched. If there are no
objections, I'll take care of that.

And done.

Thanks!

- ilmari
--
"A disappointingly low fraction of the human race is,
at any given time, on fire." - Stig Sandbeck Mathisen