Re: [PATCH] Missing links between system catalog documentation pages
Fabien COELHO <coelho@cri.ensmp.fr>
From: Fabien COELHO <coelho@cri.ensmp.fr>
To: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2020-06-21T07:01:12Z
Lists: pgsql-hackers
Hello Dagfinn, >>>> The attached patch applies cleanly, doc generation is ok. I'm ok with adding such links systematically. >>>> makes the first mention of another system catalog or view (as well as >>>> pg_hba.conf in pg_hba_file_lines) a link, for easier navigation. >> >> Why only the first mention? It seems unlikely that I would ever read >> such chapter linearly, and even so that I would want to jump especially >> on the first occurrence but not on others, so ISTM that it should be >> done all mentions? > > It's the first mention in the introductory paragraph of _each_ catalog > table/view page, not the first mention in the entire catalogs.sgml file. > E.g. https://www.postgresql.org/docs/current/catalog-pg-aggregate.html > has two mentions of pg_proc one word apart: > > Each entry in pg_aggregate is an extension of an entry in pg_proc. The > pg_proc entry carries the aggregate's name, … > > I didn't think there was much point in linkifying both in that case, and > other similar situations. The point is that the user reads a sentence, attempts to jump but sometimes can't, because the is not the first occurrence. I'd go for all mentions of another relation should be link. Alse, ISTM you missed some, maybe you could consider adding them? eg pg_database in the very first paragraph of the file, pg_attrdef in pg_attribute description, quite a few in pg_class… -- Fabien.
Commits
-
doc: Add missing cross-links in system catalog documentation
- cb6eb4a09e8a 14.0 landed
-
doc: Make SQL command names in the catalog documentation links
- 5ccf32211832 14.0 landed