aboutsummaryrefslogtreecommitdiff
path: root/src/backend/parser/parse_utilcmd.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2020-05-14 11:10:31 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2020-05-14 11:10:31 -0400
commit29c3e2dd5a6aeaf1a23d7d83d665501e2dcc6955 (patch)
treed855391c63c8464375e49e962c3cc3ee9e171da6 /src/backend/parser/parse_utilcmd.c
parent07451e1f1adc4ff832196f1f47def13e49d2ed38 (diff)
downloadpostgresql-29c3e2dd5a6aeaf1a23d7d83d665501e2dcc6955.tar.gz
postgresql-29c3e2dd5a6aeaf1a23d7d83d665501e2dcc6955.zip
Collect built-in LWLock tranche names statically, not dynamically.
There is little point in using the LWLockRegisterTranche mechanism for built-in tranche names. It wastes cycles, it creates opportunities for bugs (since failing to register a tranche name is a very hard-to-detect problem), and the lack of any centralized list of names encourages sloppy nonconformity in name choices. Moreover, since we have a centralized list of the tranches anyway in enum BuiltinTrancheIds, we're certainly not buying any flexibility in return for these disadvantages. Hence, nuke all the backend-internal LWLockRegisterTranche calls, and instead provide a const array of the builtin tranche names. (I have in mind to change a bunch of these names shortly, but this patch is just about getting them into one place.) Discussion: https://postgr.es/m/9056.1589419765@sss.pgh.pa.us
Diffstat (limited to 'src/backend/parser/parse_utilcmd.c')
0 files changed, 0 insertions, 0 deletions