From: Sergey Burladyan <eshkinkot@gmail.com>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: Bruce Momjian <bruce@momjian.us>,
  PostgreSQL-development <pgsql-hackers@postgresql.org>,
  Dmitriy Sarafannikov <dsarafannikov@yandex.ru>,
  Vladimir Borodin <root@simply.name>
Subject: Re: Broken hint bits (freeze)
References: <C4ADB713-CF88-4A75-A3C3-10E4E0A750DB@simply.name>
	<20170613193124.GQ13873@momjian.us> <87mv98af8j.fsf@seb.koffice.internal>
	<CAA4eK1JB=5gHNhRWgtVcqCQbUzGwR2LM9msYBQVULDfA+1ugUA@mail.gmail.com>
	<20170616164641.GP11450@momjian.us> <87fuezalc3.fsf@seb.koffice.internal>
	<CAA4eK1KMxzg_q2-0rMRh22EMWGutQ0QbCtMqc_5GeuNVyJ8kmA@mail.gmail.com>
	<20170619222113.GF3180@momjian.us>
	<CAJ2ymdg=0knii-bavVLpKEGc9DanaZC+UXjnipx_n-gFvOkdcg@mail.gmail.com>
	<20170620025919.GA28277@momjian.us> <20170620033726.GB28277@momjian.us>
	<87h8zb6kbx.fsf@seb.koffice.internal>
	<CAA4eK1JmEAj9qUMbS-=u7b4Boqsr3mAoadevT9LDWm4JgxamBg@mail.gmail.com>
Date: Tue, 20 Jun 2017 16:35:01 +0300
Message-ID: <8760fq7pfe.fsf@seb.koffice.internal>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
Sender: pgsql-hackers-owner@postgresql.org

Amit Kapila <amit.kapila16@gmail.com> writes:

> On Tue, Jun 20, 2017 at 3:40 PM, Sergey Burladyan <eshkinkot@gmail.com> w=
rote:
> > Bruce Momjian <bruce@momjian.us> writes:
> >
> >> On Mon, Jun 19, 2017 at 10:59:19PM -0400, Bruce Momjian wrote:
> >> > On Tue, Jun 20, 2017 at 03:50:29AM +0300, Sergey Burladyan wrote:
> >> > > 20 =D0=B8=D1=8E=D0=BD. 2017 =D0=B3. 1:21 =D0=BF=D0=BE=D0=BB=D1=8C=
=D0=B7=D0=BE=D0=B2=D0=B0=D1=82=D0=B5=D0=BB=D1=8C "Bruce Momjian" <bruce@mom=
jian.us> =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=D0=BB:
> >> > >
> >> > >
> >> > >     We are saying that Log-Shipping should match "Latest checkpoint
> >> > >     location", but the WAL for that will not be sent to the standb=
y, so it
> >> > >     will not match, but that is OK since the only thing in the non=
-shipped
> >> > >     WAL file is the checkpoint record.  How should we modify the w=
ording on
> >> > >     this?
> >> > >
> >> > >
> >> > > I am afraid that without this checkpoint record standby cannot make
> >> > > restartpoint
> >> > > and without restartpoint it does not sync shared buffers into disk=
 at
> >> > > shutdown.
> >> >
>
> It seems to me at shutdown time on standby servers we specifically
> make restart points.  See below code in ShutdownXLOG()
>
> ..
> if (RecoveryInProgress())
> CreateRestartPoint(CHECKPOINT_IS_SHUTDOWN | CHECKPOINT_IMMEDIATE);
> ..
>
> Do you have something else in mind?

What buffers this restartpoint will save into disk? I think what it can
save only buffers with LSN lower or equal to "Latest checkpoint
location". Buffers with LSN between "Minimum recovery ending
location" and "Latest checkpoint location" will not saved at all.

I set log_min_messages=3Ddebug2 and it is more clearly what happened here:
2017-06-20 13:18:32 GMT LOG:  restartpoint starting: xlog
...
2017-06-20 13:18:33 GMT DEBUG:  postmaster received signal 15
2017-06-20 13:18:33 GMT LOG:  received smart shutdown request
2017-06-20 13:18:33 GMT DEBUG:  updated min recovery point to 0/12000000
2017-06-20 13:18:33 GMT CONTEXT:  writing block 2967 of relation base/16384=
/16385
2017-06-20 13:18:33 GMT DEBUG:  checkpoint sync: number=3D1 file=3Dglobal/1=
2587 time=3D0.001 msec
2017-06-20 13:18:33 GMT DEBUG:  checkpoint sync: number=3D2 file=3Dbase/163=
84/12357 time=3D0.000 msec
2017-06-20 13:18:33 GMT DEBUG:  checkpoint sync: number=3D3 file=3Dbase/163=
84/16385 time=3D0.000 msec
2017-06-20 13:18:33 GMT DEBUG:  attempting to remove WAL segments older tha=
n log file 00000001000000000000000B
2017-06-20 13:18:33 GMT DEBUG:  recycled transaction log file "000000010000=
00000000000B"
2017-06-20 13:18:33 GMT DEBUG:  recycled transaction log file "000000010000=
00000000000A"
2017-06-20 13:18:33 GMT DEBUG:  recycled transaction log file "000000010000=
000000000009"
2017-06-20 13:18:33 GMT DEBUG:  SlruScanDirectory invoking callback on pg_s=
ubtrans/0000
2017-06-20 13:18:33 GMT LOG:  restartpoint complete: wrote 1824 buffers (44=
.5%); 0 transaction log file(s) added, 0 removed, 3 recycled; write=3D1.389=
 s, sync=3D0.000 s, total=3D1.389 s; sync files=3D3, longest=3D0.000 s, ave=
rage=3D0.000 s
2017-06-20 13:18:33 GMT LOG:  recovery restart point at 0/F008D28
2017-06-20 13:18:33 GMT DETAIL:  last completed transaction was at log time=
 2017-06-20 13:18:29.282645+00
2017-06-20 13:18:33 GMT LOG:  shutting down
2017-06-20 13:18:33 GMT DEBUG:  skipping restartpoint, already performed at=
 0/F008D28
2017-06-20 13:18:33 GMT LOG:  database system is shut down
=3D=3D=3D=3D=3D=3D=3D=3D

I use pg 9.2 and "skipping restartpoint, already performed at" is from
src/backend/access/transam/xlog.c:8643
after this statement it return from CreateRestartPoint() and do not run
   8687     CheckPointGuts(lastCheckPoint.redo, flags);

> >> > Uh, as I understand it the rsync is going to copy the missing WAL fi=
le
> >> > from the new master to the standby, right, and I think pg_controldata
> >> > too, so it should be fine.  Have you tested to see if it fails?
> >
> > It need old WAL files from old version for correct restore heap
> > files. New WAL files from new version does not have this information.
> >
>
> So in such a case can we run rsync once before pg_upgrade?

I just copy last WAL from stopped old master into running old standby
before it shutdown and wait till it replayed. After that standby can
issue restartpoint at the same location as in stopped master.

I am not sure about rsync, in my production server I have for example
111 GB in pg_xlog and if I run rsync for pg_xlog it must send ~ 40GB
of new WALs I think.

--=20
Sergey Burladyan


--=20
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers