NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit3979ff1

committed

Don't crash when pg_xlog is empty and pg_basebackup -x is used

The backup will not work (without a logarchive, and that's the wholepoint of -x) in this case, this patch just changes it to throw anerror instead of crashing when this happens.Noticed and diagnosed by TAKATSUKA Haruka

1 parentbd5ab4b commit3979ff1Copy full SHA for 3979ff1

File tree

1 file changed

-0

lines changed

src/backend/replication
- basebackup.c

1 file changed

-0

lines changed

`‎src/backend/replication/basebackup.c`

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -297,6 +297,14 @@ perform_base_backup(basebackup_options opt, DIR tblspcdir)`
`297`	`297`	`}`
`298`	`298`	`qsort(walFiles,nWalFiles,sizeof(char*),compareWalFileNames);`
`299`	`299`
	`300`	`+/*`
	`301`	`+ * There must be at least one xlog file in the pg_xlog directory,`
	`302`	`+ * since we are doing backup-including-xlog.`
	`303`	`+ */`
	`304`	`+if (nWalFiles<1)`
	`305`	`+ereport(ERROR,`
	`306`	`+(errmsg("could not find any WAL files")));`
	`307`	`+`
`300`	`308`	`/*`
`301`	`309`	`* Sanity check: the first and last segment should cover startptr and`
`302`	`310`	`* endptr, with no gaps in between.`

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!

Commit3979ff1

File tree

1 file changed

1 file changed

`‎src/backend/replication/basebackup.c`

0 commit comments