From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Make decompilation of optimized CASE constructs more robust. |
Date: | 2011-05-26 23:44:05 |
Message-ID: | E1QPkDl-0000Dn-0Z@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Make decompilation of optimized CASE constructs more robust.
We had some hacks in ruleutils.c to cope with various odd transformations
that the optimizer could do on a CASE foo WHEN "CaseTestExpr = RHS" clause.
However, the fundamental impossibility of covering all cases was exposed
by Heikki, who pointed out that the "=" operator could get replaced by an
inlined SQL function, which could contain nearly anything at all. So give
up on the hacks and just print the expression as-is if we fail to recognize
it as "CaseTestExpr = RHS". (We must cover that case so that decompiled
rules print correctly; but we are not under any obligation to make EXPLAIN
output be 100% valid SQL in all cases, and already could not do so in some
other cases.) This approach requires that we have some printable
representation of the CaseTestExpr node type; I used "CASE_TEST_EXPR".
Back-patch to all supported branches, since the problem case fails in all.
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/3987e9e62046bd800d8d08566ed49fee1ae6cb86
Modified Files
--------------
src/backend/utils/adt/ruleutils.c | 67 ++++++++++++++++++-------------------
1 files changed, 33 insertions(+), 34 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | User Okbob | 2011-05-27 09:12:27 | orafce - orafce: fixed a issues related to 9.1 collations fixed a gcc |
Previous Message | Tom Lane | 2011-05-26 21:30:02 | pgsql: Adjust configure to use "+Olibmerrno" with HP-UX C compiler, if |