Re: [HACKERS] Adding column_constraint description in ALTER TABLE synopsis

From: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
To: Stephen Frost <sfrost(at)snowman(dot)net>
Cc: Lætitia Avrot <laetitia(dot)avrot(at)gmail(dot)com>, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] Adding column_constraint description in ALTER TABLE synopsis
Date: 2018-01-22 23:57:49
Message-ID: CAEepm=3JkkbagViVZfOTTAWe55mSqKNqFdbMW+g1H2s9YztwPA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jan 23, 2018 at 12:41 PM, Thomas Munro
<thomas(dot)munro(at)enterprisedb(dot)com> wrote:
> On Mon, Jan 15, 2018 at 2:32 PM, Stephen Frost <sfrost(at)snowman(dot)net> wrote:
>> If someone else would like to review it, that'd be great, otherwise I'll
>> probably get it committed soon.
>
> FYI the v2 doesn't build:
>
> ref/alter_table.sgml:135: parser error : Opening and ending tag
> mismatch: refentry line 6 and synopsis
> </synopsis>

Here's an update to move the new stuff to the correct side of the
closing "</synopsis>". Builds for me, and the typesetting looks OK.
I'm not sure why the preexisting bogo-productions have inconsistent
indentation levels (looking at table_constraint_using_index) but
that's not this patch's fault.

--
Thomas Munro
http://www.enterprisedb.com

Attachment Content-Type Size
adding_column_constraint_description_in_ALTER_TABLE_synopsis_v3.patch application/octet-stream 3.7 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ian Barwick 2018-01-23 00:28:45 Re: Add %r substitution for psql prompts to show recovery status
Previous Message Tom Lane 2018-01-22 23:53:10 Re: pgsql: Add parallel-aware hash joins.