From f9b5b41ef993a9b76b7f97b271df8034f1a24154 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Mon, 11 Nov 2002 22:19:25 +0000 Subject: Code review for ON COMMIT patch. Make the actual on-commit action happen before commit, not after :-( --- the original coding is not only unsafe if an error occurs while it's processing, but it generates an invalid sequence of WAL entries. Resurrect 7.2 logic for deleting items when no longer needed. Use an enum instead of random macros. Editorialize on names used for routines and constants. Teach backend/nodes routines about new field in CreateTable struct. Add a regression test. --- src/backend/commands/sequence.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/backend/commands/sequence.c') diff --git a/src/backend/commands/sequence.c b/src/backend/commands/sequence.c index 31fb270c63e..f5d41b3df2c 100644 --- a/src/backend/commands/sequence.c +++ b/src/backend/commands/sequence.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/commands/sequence.c,v 1.89 2002/11/10 00:10:20 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/commands/sequence.c,v 1.90 2002/11/11 22:19:21 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -193,6 +193,7 @@ DefineSequence(CreateSeqStmt *seq) stmt->inhRelations = NIL; stmt->constraints = NIL; stmt->hasoids = false; + stmt->oncommit = ONCOMMIT_NOOP; seqoid = DefineRelation(stmt, RELKIND_SEQUENCE); -- cgit v1.2.3