Re: BUFFERS enabled by default in EXPLAIN (ANALYZE)

2022-01-17 Thread Julien Rouhaud
Hi, On Wed, Dec 01, 2021 at 06:58:20PM -0600, Justin Pryzby wrote: > > The attached patch series now looks like this (some minor patches are not > included in this list): This version of the patchset doesn't apply anymore: http://cfbot.cputube.org/patch_36_3409.log === Applying patches on top o

Re: BUFFERS enabled by default in EXPLAIN (ANALYZE)

2021-12-01 Thread Justin Pryzby
On Mon, Nov 15, 2021 at 01:09:54PM -0600, Justin Pryzby wrote: > Some time ago, I had a few relevant patches: > 1) add explain(REGRESS) which is shorthand for (BUFFERS OFF, TIMING OFF, > COSTS OFF, SUMMARY OFF) > 2) add explain(MACHINE) which elides machine-specific output from explain; >for e

Re: BUFFERS enabled by default in EXPLAIN (ANALYZE)

2021-11-24 Thread Michael Christofides
I think it *should* be enabled for planning, since that makes the default > easier to understand and document, and it makes a user's use of "explain" > easier. I’d be keen to see BUFFERS off by default with EXPLAIN, and on by default with EXPLAIN ANALYZE. The SUMMARY flag was implemented that wa

Re: BUFFERS enabled by default in EXPLAIN (ANALYZE)

2021-11-15 Thread Justin Pryzby
On Fri, Nov 12, 2021 at 02:58:07PM -0800, Nikolay Samokhvalov wrote: > Re-reading the thread [1] (cannot answer there – don't have those emails in > my box anymore), I see that there was strong support for enabling BUFFERS > in EXPLAIN ANALYZE by default. And there were patches. Commitfest entry [2

Re: BUFFERS enabled by default in EXPLAIN (ANALYZE)

2021-11-12 Thread Julien Rouhaud
On Sat, Nov 13, 2021 at 8:18 AM Tomas Vondra wrote: > > On 11/12/21 23:58, Nikolay Samokhvalov wrote: > > Re-reading the thread [1] (cannot answer there – don't have those emails > > in my box anymore), > > You can download the message as mbox and import it into your client > (pretty much any clie

Re: BUFFERS enabled by default in EXPLAIN (ANALYZE)

2021-11-12 Thread Vik Fearing
On 11/12/21 11:58 PM, Nikolay Samokhvalov wrote: > Re-reading the thread [1] (cannot answer there – don't have those emails in > my box anymore), I see that there was strong support for enabling BUFFERS > in EXPLAIN ANALYZE by default. And there were patches. Commitfest entry [2] > was marked Rejec

Re: BUFFERS enabled by default in EXPLAIN (ANALYZE)

2021-11-12 Thread Tomas Vondra
On 11/12/21 23:58, Nikolay Samokhvalov wrote: Re-reading the thread [1] (cannot answer there – don't have those emails in my box anymore), You can download the message as mbox and import it into your client (pretty much any client supports that, I think). regards -- Tomas Vondra Enterpr