NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commita9613ee

committed

Ignore recovery_min_apply_delay until recovery has reached consistent state

Previously recovery_min_apply_delay was applied even before recoveryhad reached consistency. This could cause us to wait a long timeunexpectedly for read-only connections to be allowed. It's problematicbecause the standby was useless during that wait time.This patch changes recovery_min_apply_delay so that it's applied oncethe database has reached the consistent state. That is, even if the delayis set, the standby tries to replay WAL records as fast as possible untilit has reached consistency.Author: Michael PaquierReviewed-By: Julien RouhaudReported-By: Greg CloughBackpatch: 9.4, where recovery_min_apply_delay was addedBug: #13770Discussion:http://www.postgresql.org/message-id/20151111155006.2644.84564@wrigleys.postgresql.org

1 parent2105091 commita9613eeCopy full SHA for a9613ee

File tree

2 files changed

-2

lines changed

doc/src/sgml
- recovery-config.sgml
src/backend/access/transam
- xlog.c

2 files changed

-2

lines changed

`‎doc/src/sgml/recovery-config.sgml`

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -441,8 +441,9 @@ restore_command = 'copy "C:\\server\\archivedir\\%f" "%p"' # Windows`
`441`	`441`	`are not visible until the corresponding commit record is applied.`
`442`	`442`	`</para>`
`443`	`443`	`<para>`
`444`		`- The delay occurs until the standby is promoted or triggered. After that`
`445`		`- the standby will end recovery without further waiting.`
	`444`	`+ The delay occurs once the database in recovery has reached a consistent`
	`445`	`+ state, until the standby is promoted or triggered. After that the standby`
	`446`	`+ will end recovery without further waiting.`
`446`	`447`	`</para>`
`447`	`448`	`<para>`
`448`	`449`	`This parameter is intended for use with streaming replication deployments;`

`‎src/backend/access/transam/xlog.c`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -5856,6 +5856,10 @@ recoveryApplyDelay(XLogRecord *record)`
`5856`	`5856`	`if (recovery_min_apply_delay <=0)`
`5857`	`5857`	`return false;`
`5858`	`5858`
	`5859`	`+/* no delay is applied on a database not yet consistent */`
	`5860`	`+if (!reachedConsistency)`
	`5861`	`+return false;`
	`5862`	`+`
`5859`	`5863`	`/*`
`5860`	`5864`	`* Is it a COMMIT record?`
`5861`	`5865`	`*`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commita9613ee

File tree

2 files changed

2 files changed

`‎doc/src/sgml/recovery-config.sgml`

`‎src/backend/access/transam/xlog.c`

0 commit comments