Re: Enhanced error message to include hint messages for redundant options error

From: Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>
To: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
Cc: vignesh C <vignesh21(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Enhanced error message to include hint messages for redundant options error
Date: 2021-04-27 11:43:33
Message-ID: CALj2ACWNhf7krcXUEwq3ttWwoUo3LCa0zZDo9_gKtu7kTU9aHw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Apr 27, 2021 at 6:23 AM Bharath Rupireddy
<bharath(dot)rupireddyforpostgres(at)gmail(dot)com> wrote:
>
> On Mon, Apr 26, 2021 at 9:10 PM Bharath Rupireddy
> <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> wrote:
> >
> > I found another problem with collationcmds.c is that it doesn't error
> > out if some of the options are specified more than once, something
> > like below. I think the option checking "for loop" in DefineCollation
> > needs to be reworked.
> > CREATE COLLATION case_insensitive (provider = icu, provider =
> > someother locale = '@colStrength=secondary', deterministic = false,
> > deterministic = true);
>
> I'm thinking that the above problem should be discussed separately. I
> will start a new thread soon on this.

I started a separate thread -
https://www.postgresql.org/message-id/CALj2ACWtL6fTLdyF4R_YkPtf1YEDb6FUoD5DGAki3rpD%2BsWqiA%40mail.gmail.com

With Regards,
Bharath Rupireddy.
EnterpriseDB: http://www.enterprisedb.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Dilip Kumar 2021-04-27 11:48:07 Re: [BUG] "FailedAssertion" reported when streaming in logical replication
Previous Message Andrew Dunstan 2021-04-27 11:23:16 Re: Attach to shared memory after fork()