Re: EXCLUDE COLLATE in CREATE/ALTER TABLE document

From: Shubham Khanna <khannashubham1197(at)gmail(dot)com>
To: shihao zhong <zhong950419(at)gmail(dot)com>
Cc: jian he <jian(dot)universality(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: EXCLUDE COLLATE in CREATE/ALTER TABLE document
Date: 2023-11-29 04:49:34
Message-ID: CAHv8RjJRht7Mdw4w7D7he_Q+24S-eRoc3-3K3EWVbnxkU+sADA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Nov 17, 2023 at 4:55 AM shihao zhong <zhong950419(at)gmail(dot)com> wrote:
>
> Hi Jian,
>
> Thanks for your comments, a new version is attached.
>
> Thanks,
> Shihao
>
> On Fri, Nov 10, 2023 at 9:59 AM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
>>
>> On Wed, Nov 1, 2023 at 10:30 AM shihao zhong <zhong950419(at)gmail(dot)com> wrote:
>> >
>> > Thank you for your feedback on my previous patch. I have fixed the issue and attached a new patch for your review. Could you please take a look for it if you have a sec? Thanks
>> >
>>
>> Your patch works fine. you can see it here:
>> https://cirrus-ci.com/task/6481922939944960
>> in an ideal world, since the doc is already built, we can probably
>> view it as a plain html file just click the ci test result.
>>
>> in src/sgml/ref/create_table.sgml:
>> "Each exclude_element can optionally specify an operator class and/or
>> ordering options; these are described fully under CREATE INDEX."
>>
>> You may need to update this sentence to reflect that exclude_element
>> can also optionally specify collation.

I have reviewed the changes and it looks fine.

Thanks and Regards,
Shubham Khanna.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Zhijie Hou (Fujitsu) 2023-11-29 05:58:06 RE: Synchronizing slots from primary to standby
Previous Message Tom Lane 2023-11-29 04:30:59 Re: SSL tests fail on OpenSSL v3.2.0