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:02:18 -0400 |
| commit | 68043258ac54b09565e9bebadb16214d9b6df933 (patch) | |
| tree | ca768620b18500e517c8dfc82668650d027fab09 /src/backend/utils/adt/ruleutils.c | |
| parent | d86fb72c5c7ebf71e238ac87daa0c1f8043a1717 (diff) | |
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 9ca3b9d0c44..f245e91379c 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: |