Hello, Please find below a submission of a patch to the PostgreSQL documentation.
You can also find the patch in this git commit: https://gitlab.com/paulrbr/postgresql/commit/024d3870450df6dcdc69bddbe2de46084b73e3a2.diff ============ commit 024d3870450df6dcdc69bddbe2de46084b73e3a2 Author: Paul B <paul.bon...@capitainetrain.com> Date: Thu Aug 16 18:25:22 2018 +0200 doc: Update pg_upgrade page while checking checkpoint locations At the end of the previous step 7. you already stopped the primary server. Which led PostgreSQL to issue a CHECKPOINT command. Aksi the end of the step 7. also states that "standby servers can remain running until a later step". However if you keep your standby server running at this point and compare the <quote>Latest checkpoint location</quote> with your stopped primary you will never end up with matching values. I found it confusing during my pg_upgrade tests as I tried multiple times to end up with the same latest checkpoint location value between my primary and standby nodes until I realised PostgreSQL was issuing a CHECKPOINT during shutdown which would obviously prevent that. I reckon some clarification should be added to the documentation for that and that is why I propose this patch. Please let me know if you want to phrase it differently or if I am missing something out. Thank you! diff --git a/doc/src/sgml/ref/pgupgrade.sgml b/doc/src/sgml/ref/pgupgrade.sgml index 6dafb404a11..d51146d641d 100644 --- a/doc/src/sgml/ref/pgupgrade.sgml +++ b/doc/src/sgml/ref/pgupgrade.sgml @@ -326,7 +326,8 @@ NET STOP postgresql-&majorversion; against the old primary and standby clusters. Verify that the <quote>Latest checkpoint location</quote> values match in all clusters. (There will be a mismatch if old standby servers were shut down - before the old primary.) Also, change <varname>wal_level</varname> to + before the old primary or if the old standby servers are still running.) + Also, change <varname>wal_level</varname> to <literal>replica</literal> in the <filename>postgresql.conf</filename> file on the new primary cluster. </para> ============ --- Paul Bonaud
signature.asc
Description: OpenPGP digital signature