Re: Slave enters in recovery and promotes when WAL stream with master is cut + delay master/slave
Andres Freund <andres@2ndquadrant.com>
From: Andres Freund <andres@2ndquadrant.com>
To: Heikki Linnakangas <hlinnakangas@vmware.com>
Cc: Michael Paquier <michael.paquier@gmail.com>, Alvaro Herrera <alvherre@2ndquadrant.com>, hlinnakanga@anarazel.de, PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Date: 2013-01-17T15:23:44Z
Lists: pgsql-hackers
On 2013-01-17 17:18:14 +0200, Heikki Linnakangas wrote: > On 17.01.2013 15:05, Andres Freund wrote: > >On 2013-01-17 13:47:41 +0900, Michael Paquier wrote: > >>I think that bug has been introduced by commit 7fcbf6a. > >>Before splitting xlog reading as a separate facility things worked > >>correctly. > >>There are also no delay problems before this commit. > > > >Ok, my inkling proved to be correct, its two related issues: > > > >a ) The error handling in XLogReadRecord is inconsistent, it doesn't > >always reset the necessary things. > > > >b) ReadRecord: We cannot not break out of the retry loop in readRecord > >just so, just removing the break seems correct. > > > >c) ReadRecord: (minor): We should log an error even if errormsg is not > >set, otherwise we wont jump out far enough. > > > >I think at least a) and b) is the result of merges between development > >of different people, sorry for that. > > Got a patch? Yes, I am just testing some scenarios with it, will send it after that. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
Commits
-
Use the right timeline when beginning to stream from master.
- 2ff65553131d 9.3.0 cited
-
Split out XLog reading as an independent facility
- 7fcbf6a405ff 9.3.0 cited