diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2002-11-11 22:19:25 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2002-11-11 22:19:25 +0000 |
| commit | f9b5b41ef993a9b76b7f97b271df8034f1a24154 (patch) | |
| tree | ddd95da1ffa8ce1fee1ebc807822d6cc67cd8cdd /src/backend/commands/sequence.c | |
| parent | 1b342df00af318055a1cf432c3eaa3b74347df39 (diff) | |
| download | postgresql-f9b5b41ef993a9b76b7f97b271df8034f1a24154.tar.gz postgresql-f9b5b41ef993a9b76b7f97b271df8034f1a24154.zip | |
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.
Diffstat (limited to 'src/backend/commands/sequence.c')
| -rw-r--r-- | src/backend/commands/sequence.c | 3 |
1 files changed, 2 insertions, 1 deletions
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); |