From: | Amul Sul <sulamul(at)gmail(dot)com> |
---|---|
To: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
Cc: | Prabhat Sahu <prabhat(dot)sahu(at)enterprisedb(dot)com>, Ibrar Ahmed <ibrar(dot)ahmad(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [Patch] ALTER SYSTEM READ ONLY |
Date: | 2021-04-12 14:04:15 |
Message-ID: | CAAJ_b94ExwquJfonAKGqqpRk3CBW7zSbbE5UCNsd2RzE7PboAQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Rebased again.
On Wed, Apr 7, 2021 at 12:38 PM Amul Sul <sulamul(at)gmail(dot)com> wrote:
>
> Rotten again, attached the rebased version.
>
> Regards,
> Amul
>
> On Mon, Apr 5, 2021 at 5:27 PM Amul Sul <sulamul(at)gmail(dot)com> wrote:
> >
> > On Mon, Apr 5, 2021 at 4:45 PM Bharath Rupireddy
> > <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> wrote:
> > >
> >
> > Thanks Bharath for your review.
> >
> > > On Mon, Apr 5, 2021 at 11:02 AM Amul Sul <sulamul(at)gmail(dot)com> wrote:
> > > >
> > > > Attached is the rebase version for the latest master head(commit # 9f6f1f9b8e6).
> > >
> > > Some minor comments on 0001:
> > > Isn't it "might not be running"?
> > > + errdetail("Checkpointer might not running."),
> > >
> >
> > Ok, fixed in the attached version.
> >
> > > Isn't it "Try again after sometime"?
> > > + errhint("Try after sometime again.")));
> > >
> >
> > Ok, done.
> >
> > > Can we have ereport(DEBUG1 just to be consistent(although it doesn't
> > > make any difference from elog(DEBUG1) with the new log messages
> > > introduced in the patch?
> > > + elog(DEBUG1, "waiting for backends to adopt requested WAL
> > > prohibit state change");
> > >
> >
> > I think it's fine; many existing places have used elog(DEBUG1, ....) too.
> >
> > Regards,
> > Amul
Attachment | Content-Type | Size |
---|---|---|
v26-0001-Implement-wal-prohibit-state-using-global-barrie.patch | application/octet-stream | 52.5 KB |
v26-0003-Documentation.patch | application/octet-stream | 10.3 KB |
v26-0002-Error-or-Assert-before-START_CRIT_SECTION-for-WA.patch | application/octet-stream | 69.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2021-04-12 14:25:33 | Re: psql - add SHOW_ALL_RESULTS option |
Previous Message | Konstantina Skovola | 2021-04-12 13:56:20 | [GSoC 2021 proposal] pl/julia extension |