Re: Dead reference to schema_only_with_statistics in pg_dump TAP code
Ewan Young <kdbase.hack@gmail.com>
From: Ewan Young <kdbase.hack@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2026-06-15T10:04:59Z
Lists: pgsql-hackers
Hi Michael, Confirmed here. The schema_only_with_statistics run definition was removed in 0ed92cf50cc4 (when --schema-only + --with-statistics became a rejected combination), but the references left behind in %full_runs and in the per-test like/unlike lists were never cleaned up. Since the main loop only iterates over keys %pgdump_runs, those leftover keys are never looked up, so they've simply been inert. LGTM, and +1 for including v18 to keep future back-patches quiet. Thanks, Ewan On Mon, Jun 15, 2026 at 4:07 PM Michael Paquier <michael@paquier.xyz> wrote: > > Hi all, > > While looking at the TAP tests of pg_dump today, I have bumped into a > dead reference to a command sequence named schema_only_with_statistics. > This reference has been removed in 0ed92cf50cc4, and has been listed a > couple of times after that, having no effect. > > Let's clean up that as of the attached on HEAD, at least. v18 would > be nice to include in the set, as this script is already complicated > and that could create noise when backpatching. > > Comments or opinions? > -- > Michael
Commits
-
pg_dump: Remove dead code in TAP tests
- 54ffa74c9924 18 (unreleased) landed
- d3b345eef3f3 19 (unreleased) landed