On 10.06.23 06:00, Andres Freund wrote:
 From c5e637a54c2b83e5bd8c4155784d97e82937eb51 Mon Sep 17 00:00:00 2001
From: Samay Sharma<smilingsa...@gmail.com>
Date: Mon, 6 Feb 2023 16:09:42 -0800
Subject: [PATCH v9 2/5] Add data layout options sub-section in installation
  docs

This commit separates out blocksize, segsize and wal_blocksize
options into a separate Data layout options sub-section in both
the make and meson docs. They were earlier in a miscellaneous
section which included several unrelated options. This change
also helps reduce the repetition of the warnings that changing
these parameters breaks on-disk compatibility.
I still like this change, but ISTM that the "Data Layout" section should
follow the "PostgreSQL Features" section, rather than follow "Anti Features",
"Build Process Details" and "Miscellaneous". I realize some of these are
reorganized later on, but even then "Build Process Details"

Would anybody mind if I swapped these around?

I don't mind a Data Layout section in principle, but I wonder whether it's worth changing now. The segsize option is proposed to be turned into a run-time option (and/or removed). For the WAL block size, I had previously mentioned, I don't think it is correct that pg_upgrade should actually care about it. So I wouldn't spend too much time trying to carefully refactor the notes on the data layout options if we're going to have to change them around before long again anyway.




Reply via email to