| 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
| 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 |