public inbox for [email protected]
help / color / mirror / Atom feedFrom: David Rowley <[email protected]>
To: Tom Lane <[email protected]>
Cc: PostgreSQL Developers <[email protected]>
Subject: Re: Does anyone ever use OPTIMIZER_DEBUG?
Date: Mon, 9 Oct 2023 10:26:38 +1300
Message-ID: <CAApHDvpzRrmzA2nv0N_ehMD1ajDQto+EpjLzfWoOW=qVu6eRRw@mail.gmail.com> (raw)
In-Reply-To: <CAApHDvoMLP3ZpAE4_zivkT1GJU293BT7Yy-Kmo-kxU+Nc_-EWg@mail.gmail.com>
References: <CAApHDvoCdjo8Cu2zEZF4-AxWG-90S+pYXAnoDDa9J3xH-OrczQ@mail.gmail.com>
<[email protected]>
<CAApHDvoMLP3ZpAE4_zivkT1GJU293BT7Yy-Kmo-kxU+Nc_-EWg@mail.gmail.com>
On Tue, 3 Oct 2023 at 12:29, David Rowley <[email protected]> wrote:
>
> On Fri, 29 Sept 2023 at 10:59, Tom Lane <[email protected]> wrote:
> > We could also discuss keeping the "tracing" aspect of it, but
> > replacing debug_print_rel with pprint(rel), which'd still allow
> > removal of all the "DEBUG SUPPORT" stuff at the bottom of allpaths.c.
> > That's pretty much all of the maintenance-requiring stuff in it.
>
> To assist discussion, I've attached a patch for that.
It looks like nobody is objecting to this. I understand that not
everyone who might object will have read this email thread, so what I
propose to do here is move along and just commit the patch to swap out
debug_print_rel and use pprint instead. If that's done now then there
are around 10 months where we could realistically revert this again if
someone were to come forward with an objection.
Sound ok?
David
view thread (4+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: Does anyone ever use OPTIMIZER_DEBUG?
In-Reply-To: <CAApHDvpzRrmzA2nv0N_ehMD1ajDQto+EpjLzfWoOW=qVu6eRRw@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox