]> granicus.if.org Git - postgresql/commit
Fix pg_dump to not emit invalid SQL for an empty operator class.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 26 May 2017 16:51:06 +0000 (12:51 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 26 May 2017 16:51:06 +0000 (12:51 -0400)
commitf62e1eff524ab9927cbadda07060cd0b30b1fc72
tree6d52bc8109646a21c24bbcba96c0e52f8f67e8ee
parent8f6396470a9d0308e4f2832eb8ba313e5aaef57d
Fix pg_dump to not emit invalid SQL for an empty operator class.

If an operator class has no operators or functions, and doesn't need
a STORAGE clause, we emitted "CREATE OPERATOR CLASS ... AS ;" which
is syntactically invalid.  Fix by forcing a STORAGE clause to be
emitted anyway in this case.

(At some point we might consider changing the grammar to allow CREATE
OPERATOR CLASS without an opclass_item_list.  But probably we'd want to
omit the AS in that case, so that wouldn't fix this pg_dump issue anyway.)

It's been like this all along, so back-patch to all supported branches.

Daniel Gustafsson, tweaked by me to avoid a dangling-pointer bug

Discussion: https://postgr.es/m/D9E5FC64-7A37-4F3D-B946-7E4FB468F88A@yesql.se
src/bin/pg_dump/pg_dump.c