diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-05-20 12:20:47 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-05-20 12:20:47 -0400 |
| commit | e416830a296f440acd67afd8599dde5411021fc4 (patch) | |
| tree | c15a617415521d3dba027c5b85cc1f4058f12f08 /src/backend/utils/adt/formatting.c | |
| parent | a0841ecd2518d4505b96132b764b918ab5d21ad4 (diff) | |
| download | postgresql-e416830a296f440acd67afd8599dde5411021fc4.tar.gz postgresql-e416830a296f440acd67afd8599dde5411021fc4.zip | |
Prevent auto_explain from changing the output of a user's EXPLAIN.
Commit af7914c6627bcf0b0ca614e9ce95d3f8056602bf, which introduced the
EXPLAIN (TIMING) option, for some reason coded explain.c to look at
planstate->instrument->need_timer rather than es->timing to decide
whether to print timing info. However, the former flag might get set
as a result of contrib/auto_explain wanting timing information. We
certainly don't want activation of auto_explain to change user-visible
statement behavior, so fix that.
Also fix an independent bug introduced in the same patch: in the code
path for a never-executed node with a machine-friendly output format,
if timing was selected, it would fail to print the Actual Rows and Actual
Loops items.
Per bug #10404 from Tomonari Katsumata. Back-patch to 9.2 where the
faulty code was introduced.
Diffstat (limited to 'src/backend/utils/adt/formatting.c')
0 files changed, 0 insertions, 0 deletions