]> granicus.if.org Git - postgresql/commit
Remove manual breaks in NodeTag assignments to fix duplicate tag numbers.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 29 Dec 2016 21:57:41 +0000 (16:57 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 29 Dec 2016 21:57:41 +0000 (16:57 -0500)
commit80a7298b9eb7f108ef20be6ee00d9513a43c61a0
tree0034216919530196afd920cbcb3178f59007db4b
parentdb779d941e0a05ea4647455d0b386b38ef58983b
Remove manual breaks in NodeTag assignments to fix duplicate tag numbers.

Commit f0e44751d added new node tags at a place in the tag numbering
where there was no daylight left before the next hard-coded number,
resulting in some duplicate tag assignments.  This doesn't seem to have
caused any big problem so far, but it's surely trouble waiting to happen.

We could adjust the manually assigned breakpoints to make more room,
but that just leaves the same hazard waiting to strike again in future.
What seems like a better idea is to get rid of the manual assignments
and leave NodeTags to be automatically assigned, consecutively from one
on up.  This means that any change in the tag list forces a backend-wide
recompile, but realistically that's usually needed anyway.

Discussion: https://postgr.es/m/29670.1482942811@sss.pgh.pa.us
src/include/nodes/nodes.h