pgsql: Refactor init_params() in sequence.c to not use FormData_pg_sequ - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Refactor init_params() in sequence.c to not use FormData_pg_sequ
Date
Msg-id E1unpnD-000PrG-2V@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Refactor init_params() in sequence.c to not use FormData_pg_sequence_data

init_params() sets up "last_value" and "is_called" for a sequence
relation holdind its metadata, based on the sequence properties in
pg_sequences.  "log_cnt" is the third property that can be updated in
this routine for FormData_pg_sequence_data, tracking when WAL records
should be generated for a sequence after nextval() iterations.  This
routine is called when creating or altering a sequence.

This commit refactors init_params() to not depend anymore on
FormData_pg_sequence_data, removing traces of it in sequence.c, making
easier the manipulation of metadata related to sequences.  The knowledge
about "log_cnt" is replaced with a more general "reset_state" flag, to
let the caller know if the sequence state should be reset.  In the case
of in-core sequences, this relates to WAL logging.  We still need to
depend on FormData_pg_sequence.

Author: Michael Paquier <michael@paquier.xyz>
Reviewed-by: Kirill Reshke <reshkekirill@gmail.com>
Reviewed-by: Tomas Vondra <tomas@vondra.me>
Discussion: https://postgr.es/m/ZWlohtKAs0uVVpZ3@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/ba3d93b2e806a877f26922e0f9e1845d0ef1511b

Modified Files
--------------
src/backend/commands/sequence.c | 81 +++++++++++++++++++++++++----------------
1 file changed, 49 insertions(+), 32 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Remove md5() call from isolation test for CLUSTER and TOAST
Next
From: Peter Eisentraut
Date:
Subject: pgsql: meson: Move C99 test earlier