| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | pgsql-committers(at)postgresql(dot)org |
| Subject: | pgsql: Fix ALTER OPERATOR to update dependencies properly. |
| Date: | 2015-12-31 22:37:37 |
| Message-ID: | E1aElqj-0004r2-Cc@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Fix ALTER OPERATOR to update dependencies properly.
Fix an oversight in commit 321eed5f0f7563a0: replacing an operator's
selectivity functions needs to result in a corresponding update in
pg_depend. We have a function that can handle that, but it was not
called by AlterOperator().
To fix this without enlarging pg_operator.h's #include list beyond
what clients can safely include, split off the function definitions
into a new file pg_operator_fn.h, similarly to what we've done for
some other catalog header files. It's not entirely clear whether
any client-side code needs to include pg_operator.h, but it seems
prudent to assume that there is some such code somewhere.
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/0dab5ef39b3d9d86e45bbbb2f6ea60b4f5517d9a
Modified Files
--------------
src/backend/catalog/pg_operator.c | 35 +++++++-----
src/backend/commands/operatorcmds.c | 5 +-
src/include/catalog/pg_operator.h | 17 ------
src/include/catalog/pg_operator_fn.h | 34 ++++++++++++
src/test/regress/expected/alter_operator.out | 73 +++++++++++++++++++++++---
src/test/regress/sql/alter_operator.sql | 41 +++++++++++++--
6 files changed, 163 insertions(+), 42 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2015-12-31 22:59:24 | pgsql: Add a comment noting that FDWs don't have to implement EXCEPT or |
| Previous Message | Bruce Momjian | 2015-12-31 21:41:46 | Re: pgsql: Further tweaking of print_aligned_vertical(). |