[COMMITTERS] pgsql: Remove manual breaks in NodeTag assignments to fix duplicatetag - Mailing list pgsql-committers

From Tom Lane
Subject [COMMITTERS] pgsql: Remove manual breaks in NodeTag assignments to fix duplicatetag
Date
Msg-id E1cMiho-0006jN-KQ@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/80a7298b9eb7f108ef20be6ee00d9513a43c61a0

Modified Files
--------------
src/include/nodes/nodes.h | 30 +++++++++++++++---------------
1 file changed, 15 insertions(+), 15 deletions(-)


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: [COMMITTERS] pgsql: Expand ad-hoc unit abbreviations in function descriptions
Next
From: Tom Lane
Date:
Subject: [COMMITTERS] pgsql: Fix incorrect example of to_timestamp() usage.