From: | "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com> |
---|---|
To: | shveta malik <shveta(dot)malik(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | Peter Smith <smithpb2250(at)gmail(dot)com>, Ajin Cherian <itsajin(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Nisha Moond <nisha(dot)moond412(at)gmail(dot)com>, "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Subject: | RE: Synchronizing slots from primary to standby |
Date: | 2024-02-09 04:30:04 |
Message-ID: | OS0PR01MB5716DF37FDCBC450B59A91CE944B2@OS0PR01MB5716.jpnprd01.prod.outlook.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thursday, February 8, 2024 7:07 PM shveta malik <shveta(dot)malik(at)gmail(dot)com>
>
> On Thu, Feb 8, 2024 at 4:03 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> >
> > Few comments on 0001
> > ===================
> > 7.
> > + /*
> > + * The primary_slot_name is not set yet or WALs not received yet.
> > + * Synchronization is not possible if the walreceiver is not started.
> > + */
> > + latestWalEnd = GetWalRcvLatestWalEnd();
> > + SpinLockAcquire(&WalRcv->mutex); if ((WalRcv->slotname[0] == '\0')
> > + ||
> > + XLogRecPtrIsInvalid(latestWalEnd))
> > + {
> > + SpinLockRelease(&WalRcv->mutex);
> > + return false;
> >
> > For the purpose of 0001, we should give WARNING here.
Fixed.
Here is the V82 patch set which includes the following changes:
0001
1. Fixed one miss that the size of shared memory for slot sync was not counted
in CalculateShmemSize().
2. Added a warning message if walreceiver has not started yet.
2. Fixed the above comment.
0002 - 0003
Rebased
0004
1. Added more details that user should run second query on standby after the
primary is down.
2. Mentioned that the query needs to be run on the db that includes the failover subscription.
Thanks Shveta for working on the changes.
Best Regards,
Hou zj
Attachment | Content-Type | Size |
---|---|---|
v82-0004-Document-the-steps-to-check-if-the-standby-is-re.patch | application/octet-stream | 7.0 KB |
v82-0001-Add-a-slot-synchronization-function.patch | application/octet-stream | 72.3 KB |
v82-0002-Add-a-new-slotsync-worker.patch | application/octet-stream | 60.1 KB |
v82-0003-Allow-logical-walsenders-to-wait-for-the-physica.patch | application/octet-stream | 42.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2024-02-09 04:34:42 | Re: Synchronizing slots from primary to standby |
Previous Message | Peter Smith | 2024-02-09 04:26:43 | Re: Synchronizing slots from primary to standby |