diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-10-15 18:50:13 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-10-15 18:50:13 -0400 |
| commit | 90063a7612e2730f7757c2a80ba384bbe7e35c4b (patch) | |
| tree | 2e106aa9065c7f8f5e9e1937252118a19c64da9d /contrib/auto_explain | |
| parent | 076d29a1eed5fe51fb2b25b98fcde9dd7c506902 (diff) | |
| download | postgresql-90063a7612e2730f7757c2a80ba384bbe7e35c4b.tar.gz | |
Print planning time only in EXPLAIN ANALYZE, not plain EXPLAIN.
We've gotten enough push-back on that change to make it clear that it
wasn't an especially good idea to do it like that. Revert plain EXPLAIN
to its previous behavior, but keep the extra output in EXPLAIN ANALYZE.
Per discussion.
Internally, I set this up as a separate flag ExplainState.summary that
controls printing of planning time and execution time. For now it's
just copied from the ANALYZE option, but we could consider exposing it
to users.
Diffstat (limited to 'contrib/auto_explain')
| -rw-r--r-- | contrib/auto_explain/auto_explain.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/contrib/auto_explain/auto_explain.c b/contrib/auto_explain/auto_explain.c index b33b2211ca..d5fa6ac2d3 100644 --- a/contrib/auto_explain/auto_explain.c +++ b/contrib/auto_explain/auto_explain.c @@ -301,6 +301,7 @@ explain_ExecutorEnd(QueryDesc *queryDesc) es.verbose = auto_explain_log_verbose; es.buffers = (es.analyze && auto_explain_log_buffers); es.timing = (es.analyze && auto_explain_log_timing); + es.summary = es.analyze; es.format = auto_explain_log_format; ExplainBeginOutput(&es); |