aboutsummaryrefslogtreecommitdiff
path: root/contrib/postgres_fdw/postgres_fdw.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2015-03-15 15:19:04 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2015-03-15 15:19:04 -0400
commit9fac5fd741ec17ae24dde6b8e82064f13c148ddf (patch)
tree6822fe84eccbe2760c4bba46c1aea1399a00064c /contrib/postgres_fdw/postgres_fdw.c
parent80089597730f67927293c410914f3e6bf11ca447 (diff)
downloadpostgresql-9fac5fd741ec17ae24dde6b8e82064f13c148ddf.tar.gz
postgresql-9fac5fd741ec17ae24dde6b8e82064f13c148ddf.zip
Move LockClauseStrength, LockWaitPolicy into new file nodes/lockoptions.h.
Commit df630b0dd5ea2de52972d456f5978a012436115e moved enum LockWaitPolicy into its very own header file utils/lockwaitpolicy.h, which does not seem like a great idea from here. First, it's still a node-related declaration, and second, a file named like that can never sensibly be used for anything else. I do not think we want to encourage a one-typedef-per-header-file approach. The upcoming foreign table inheritance patch was doubling down on this bad idea by moving enum LockClauseStrength into its *own* can-never-be-used-for-anything-else file. Instead, let's put them both in a file named nodes/lockoptions.h. (They do seem to need a separate header file because we need them in both parsenodes.h and plannodes.h, and we don't want either of those including the other. Past practice might suggest adding them to nodes/nodes.h, but they don't seem sufficiently globally useful to justify that.) Committed separately since there's no functional change here, just some header-file refactoring.
Diffstat (limited to 'contrib/postgres_fdw/postgres_fdw.c')
0 files changed, 0 insertions, 0 deletions