diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2012-07-24 15:49:54 -0400 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2012-07-24 16:01:32 -0400 |
| commit | d7b47e515530520da9564b05991bd8a8c6f52b06 (patch) | |
| tree | feeada7a3a64415b1e1fe1bc50e008fe5a6329e1 /src/backend/utils/adt/ruleutils.c | |
| parent | d61d9aa7501f31f99ee089f8b014161254eafa89 (diff) | |
| download | postgresql-d7b47e515530520da9564b05991bd8a8c6f52b06.tar.gz postgresql-d7b47e515530520da9564b05991bd8a8c6f52b06.zip | |
Change syntax of new CHECK NO INHERIT constraints
The initially implemented syntax, "CHECK NO INHERIT (expr)" was not
deemed very good, so switch to "CHECK (expr) NO INHERIT" instead. This
way it looks similar to SQL-standards compliant constraint attribute.
Backport to 9.2 where the new syntax and feature was introduced.
Per discussion.
Diffstat (limited to 'src/backend/utils/adt/ruleutils.c')
| -rw-r--r-- | src/backend/utils/adt/ruleutils.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index ec93149f3e7..412dfe6f9aa 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -1343,10 +1343,9 @@ pg_get_constraintdef_worker(Oid constraintId, bool fullCommand, * Note that simply checking for leading '(' and trailing ')' * would NOT be good enough, consider "(x > 0) AND (y > 0)". */ - appendStringInfo(&buf, "CHECK %s(%s)", - conForm->connoinherit ? "NO INHERIT " : "", - consrc); - + appendStringInfo(&buf, "CHECK (%s)%s", + consrc, + conForm->connoinherit ? " NO INHERIT" : ""); break; } case CONSTRAINT_TRIGGER: |