From: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Amul Sul <sulamul(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Refactor "mutually exclusive options" error reporting code in parse_subscription_options |
Date: | 2021-07-07 02:30:20 |
Message-ID: | CALj2ACX1q-0W8wyUk0YqobCoBeXDDM+AJpBmgcnv-6vtqbLgvg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Jul 6, 2021 at 9:24 PM Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
>
> On 2021-Jul-06, Bharath Rupireddy wrote:
>
> > Thanks, Amit. I'm posting the 0002 patch which removes extra ereport
> > calls using local variables. Please review it.
>
> I looked at this the other day and I'm not sure I like it very much.
> It's not making anything any simpler, it's barely saving two lines of
> code. I think we can do without this change.
Just for the records. I will withdraw the 0002 patch as no one has
shown interest. Thanks.
Regards,
Bharath Rupireddy.
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2021-07-07 02:44:51 | Re: [PATCH] expand the units that pg_size_pretty supports on output |
Previous Message | Gurjeet Singh | 2021-07-07 02:23:54 | Warn if initdb's --sync-only option is mixed with other options |