minor tweak to catalogs.sgml pg_class.reltablespace

Started by Alvaro Herreraabout 2 years ago3 messageshackers
Jump to latest
#1Alvaro Herrera
alvherre@2ndquadrant.com

While reviewing the patch for SET ACCESS METHOD[1]/messages/by-id/202403191013.w2kr7wqlamqz@alvherre.pgsql I noticed that
pg_class.relam is not documented fully for partitioned tables, so I
proposed the attached. Also, I remove a comment that merely repeats
what was already said a few lines above.

This is intended for backpatch to 12.

[1]: /messages/by-id/202403191013.w2kr7wqlamqz@alvherre.pgsql

--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/

Attachments:

0001-Review-for-wording-on-tablespaces-on-partitioned-tab.patchtext/x-diff; charset=utf-8Download+5-8
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Alvaro Herrera (#1)
Re: minor tweak to catalogs.sgml pg_class.reltablespace

Alvaro Herrera <alvherre@alvh.no-ip.org> writes:

While reviewing the patch for SET ACCESS METHOD[1] I noticed that
pg_class.relam is not documented fully for partitioned tables, so I
proposed the attached.

The bit about "(Not meaningful if the relation has no on-disk file.)"
is not correct, and now it's adjacent to text that contradicts it.
Maybe more like

The tablespace in which this relation is stored.
If zero, the database's default tablespace is implied.
Not meaningful if the relation has no on-disk file,
except for partitioned tables, where this is the tablespace
in which partitions will be created when one is not
specified in the creation command.

regards, tom lane

#3Alvaro Herrera
alvherre@2ndquadrant.com
In reply to: Tom Lane (#2)
Re: minor tweak to catalogs.sgml pg_class.reltablespace

On 2024-Mar-19, Tom Lane wrote:

The bit about "(Not meaningful if the relation has no on-disk file.)"
is not correct, and now it's adjacent to text that contradicts it.
Maybe more like

The tablespace in which this relation is stored.
If zero, the database's default tablespace is implied.
Not meaningful if the relation has no on-disk file,
except for partitioned tables, where this is the tablespace
in which partitions will be created when one is not
specified in the creation command.

I like that wording, thanks, pushed like that.

--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
"Right now the sectors on the hard disk run clockwise, but I heard a rumor that
you can squeeze 0.2% more throughput by running them counterclockwise.
It's worth the effort. Recommended." (Gerry Pourwelle)