Re: jit and explain nontext - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: jit and explain nontext
Date
Msg-id a56f853c-e671-c4b1-7e0e-a1f0a18621e4@enterprisedb.com
Whole thread Raw
In response to Re: jit and explain nontext  (Justin Pryzby <pryzby@telsasoft.com>)
List pgsql-hackers
On 2020-10-17 21:21, Justin Pryzby wrote:
> Added at:https://commitfest.postgresql.org/30/2766/
> 
> diff --git a/src/backend/commands/explain.c b/src/backend/commands/explain.c
> index 41317f1837..7345971507 100644
> --- a/src/backend/commands/explain.c
> +++ b/src/backend/commands/explain.c
> @@ -839,7 +839,8 @@ ExplainPrintJIT(ExplainState *es, int jit_flags, JitInstrumentation *ji)
>       instr_time    total_time;
>   
>       /* don't print information if no JITing happened */
> -    if (!ji || ji->created_functions == 0)
> +    if (!ji || (ji->created_functions == 0 &&
> +            es->format == EXPLAIN_FORMAT_TEXT))
>           return;
>   
>       /* calculate total time */

Can you show an output example of where this patch makes a difference? 
Just from reading the description, I would expect some kind of 
additional JIT-related output from something like

EXPLAIN (FORMAT YAML) SELECT 1;

but I don't see anything.




pgsql-hackers by date:

Previous
From: John Naylor
Date:
Subject: Re: Should we document IS [NOT] OF?
Next
From: Robert Haas
Date:
Subject: Re: xid wraparound danger due to INDEX_CLEANUP false