Re: Combine pg_walinspect till_end_of_wal functions with others
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Cc: Julien Rouhaud <rjuju123@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2023-03-15T01:02:23Z
Lists: pgsql-hackers
On Tue, Mar 14, 2023 at 10:35:43AM +0530, Bharath Rupireddy wrote: > My thoughts are simple here - how would one (an end user, not me and > not you) figure out how to get info/stats till the end of WAL? I'm > sure it would be difficult to find that out without looking at the > code or commit history. Search for till end of WAL behaviour with new > version will be more given the 1.0 version has explicit functions to > do that. IMO, there's no harm in being explicit in how to achieve till > end of WAL functionality around in the docs. Okay. I have kept these notes, but tweaked the wording to be a bit cleaner, replacing the term "till" by "until". To my surprise while studying this point, "till" is a term older than "until" in English literacy, but it is rarely used. > I get it. I divided the patches to 0001 and 0002 with 0001 focussing > on the change of behaviour around future end LSNs, dropping till end > of WAL functions and tests tweakings related to it. 0002 has all other > tests tidy up things. > > Please find the attached v8 patch set for further review. The tests of 0001 were still too complex IMO. The changes can be much simpler as it requires only to move the till_end_of_wal() calls from pg_walinspect.sql to oldextversions.sql. Nothing more. -- Michael
Commits
-
Improve a bit the tests of pg_walinspect
- 4f1882b96059 16.0 landed
-
Tweak regression test of pg_walinspect to be less collation-sensitive
- e643a315fc58 16.0 landed
-
Rework design of functions in pg_walinspect
- 5c1b6628075a 16.0 landed
-
Refactor and improve tests of pg_walinspect
- 1f282c24e464 16.0 landed
-
doc: Fix description of pg_get_wal_stats_till_end_of_wal() in pg_walinspect
- 019f8624664d 16.0 cited