NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf50200c

committed

Silence "missing contrecord" error.

Commitdd38ff2 added a new error message "missing contrecord" whenwe fail to reassemble a record. Unfortunately that caused noisymessages to be logged by pg_waldump at end of segment, and by walsenderwhen asked to shut down on a segment boundary.Remove the new error message, so that this condition signals end-of-WAL without a message. It's arguably a reportable condition that shouldnot be silenced while performing crash recovery, but fixing that withoutintroducing noise in the other cases will require more research.Back-patch to 15.Reported-by: Tomas Vondra <tomas.vondra@enterprisedb.com>Discussion:https://postgr.es/m/6a1df56e-4656-b3ce-4b7a-a9cb41df8189%40enterprisedb.com

1 parent7ae4e78 commitf50200cCopy full SHA for f50200c

File tree

1 file changed

-8

lines changed

src/backend/access/transam
- xlogreader.c

1 file changed

-8

lines changed

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

Lines changed: 4 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -912,15 +912,11 @@ XLogDecodeNextRecord(XLogReaderState *state, bool nonblocking)`
`912`	`912`	`state->missingContrecPtr=targetPagePtr;`
`913`	`913`
`914`	`914`	`/*`
`915`		`- * If we got here without reporting an error, report one now so that`
`916`		`- * XLogPrefetcherReadRecord() doesn't bring us back a second time and`
`917`		`- * clobber the above state. Otherwise, the existing error takes`
`918`		`- * precedence.`
	`915`	`+ * If we got here without reporting an error, make sure an error is`
	`916`	`+ * queued so that XLogPrefetcherReadRecord() doesn't bring us back a`
	`917`	`+ * second time and clobber the above state.`
`919`	`918`	`*/`
`920`		`-if (!state->errormsg_buf[0])`
`921`		`-report_invalid_record(state,`
`922`		`-"missing contrecord at %X/%X",`
`923`		`-LSN_FORMAT_ARGS(RecPtr));`
	`919`	`+state->errormsg_deferred= true;`
`924`	`920`	`}`
`925`	`921`
`926`	`922`	`if (decoded&&decoded->oversized)`

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!

Commitf50200c

File tree

1 file changed

1 file changed

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

0 commit comments