pgsql: Fix incorrect logic for excluding range constructor functions in

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix incorrect logic for excluding range constructor functions in
Date: 2016-09-23 17:49:59
Message-ID: E1bnUbn-0003GU-9a@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix incorrect logic for excluding range constructor functions in pg_dump.

Faulty AND/OR nesting in the WHERE clause of getFuncs' SQL query led to
dumping range constructor functions if they are part of an extension
and we're in binary-upgrade mode. Actually, we don't want to dump them
separately even then, since CREATE TYPE AS RANGE will create the range's
constructor functions regardless. Per report from Andrew Dunstan.

It looks like this mistake was introduced by me, in commit b985d4877, in
perhaps-overzealous refactoring to reduce code duplication. I'm suitably
embarrassed.

Report: <34854939-02d7-f591-5677-ce2994104599(at)dunslane(dot)net>

Branch
------
REL9_6_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/7e02476f337990e68c80b7a1395dcc6af111c034

Modified Files
--------------
src/bin/pg_dump/pg_dump.c | 53 +++++++++++++++++++++++++++--------------------
1 file changed, 30 insertions(+), 23 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2016-09-23 18:22:51 pgsql: Doc: fix examples of # operators so they actually work.
Previous Message Tom Lane 2016-09-23 14:45:03 pgsql: Remove useless code.