Re: Dead reference to schema_only_with_statistics in pg_dump TAP code

From: Ewan Young <kdbase(dot)hack(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Dead reference to schema_only_with_statistics in pg_dump TAP code
Date: 2026-06-15 10:04:59
Message-ID: CAON2xHPrebi45msJB9Dd1ghz-yNpNZ5xvHu70Mt4hYgc1im+Ug@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
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(at)paquier(dot)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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Shlok Kyal 2026-06-15 10:13:06 Re: Support EXCEPT for ALL SEQUENCES publications
Previous Message Matheus Alcantara 2026-06-15 09:46:06 Re: LLVM JIT: any JIT-compiled query crashes (SIGILL) on a libLLVM 19 + ASAN build