| From: | Lætitia Avrot <laetitia(dot)avrot(at)gmail(dot)com> |
|---|---|
| To: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Adding column_constraint description in ALTER TABLE synopsis |
| Date: | 2017-10-30 21:13:43 |
| Message-ID: | CAB_COdgOoA=G18RhWPoW8zZ+xOxTns7xD7psHA=ct+XccokbUg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello,
As Amit Langot pointed out, the column_constraint definition is missing
whereas it is used in ALTER TABLE synopsis. It can be easily found in the
CREATE TABLE synopsis, but it's not very user friendly.
I simply copied/paste the column_constraint definition from the CREATE
TABLE synopsis to the ALTER TABLE synopsis. I also had to change the first
word "where" to "and" as it's not the first definition in that synopsis.
I choose to add it above table_constraint as column_constraint is used
before table_constraint.
The patch should apply to MASTER (or I messed up with git). I built and
tested it successfully on my laptop.
You will find enclosed my patch.
Regards,
Lætitia
| Attachment | Content-Type | Size |
|---|---|---|
| adding_column_constraint_description_in_ALTER_TABLE_synopsis_v1.patch | text/x-patch | 1.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | srielau | 2017-10-30 21:42:40 | Re: proposal: schema variables |
| Previous Message | Tomas Vondra | 2017-10-30 20:51:26 | Re: WIP: BRIN multi-range indexes |