Re: [HACKERS] Adding column_constraint description in ALTER TABLEsynopsis - Mailing list pgsql-hackers

From Amit Langote
Subject Re: [HACKERS] Adding column_constraint description in ALTER TABLEsynopsis
Date
Msg-id 27ec4df3-d1ab-3411-f87f-647f944897e1@lab.ntt.co.jp
Whole thread Raw
In response to Re: [HACKERS] Adding column_constraint description in ALTER TABLE synopsis  (Thomas Munro <thomas.munro@enterprisedb.com>)
Responses Re: [HACKERS] Adding column_constraint description in ALTER TABLEsynopsis
List pgsql-hackers
On 2018/01/23 8:57, Thomas Munro wrote:
> On Tue, Jan 23, 2018 at 12:41 PM, Thomas Munro
> <thomas.munro@enterprisedb.com> wrote:
>> On Mon, Jan 15, 2018 at 2:32 PM, Stephen Frost <sfrost@snowman.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.

Thanks for fixing that.

I noticed that partition_bound_spec in the patch is missing hash partition
bound syntax that was added after the original patch was written.  Fixed
in the attached.

Thanks,
Amit

Attachment

pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: [HACKERS] Patch: Add --no-comments to skip COMMENTs with pg_dump
Next
From: Michael Paquier
Date:
Subject: Re: [HACKERS] Deadlock in XLogInsert at AIX