NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitc943e2a

committed

Fix pg_rewind debug output to print the source timeline history

getTimelineHistory() is called twice, to read the source and thetarget timeline history files. However, the loop to print the filewith the --debug option used the wrong variable when dealing with thesource. As a result, the source's history was always printed as empty.Spotted while debugging bug #18575, but this does not fix that bug,just the debugging output. Backpatch to all supported versions.Discussion:https://www.postgresql.org/message-id/092dd515-b7b4-4fd0-8407-ceca2f02f6ec@iki.fi

1 parentf0096ef commitc943e2aCopy full SHA for c943e2a

File tree

1 file changed

-4

lines changed

src/bin/pg_rewind
- pg_rewind.c

1 file changed

-4

lines changed

`‎src/bin/pg_rewind/pg_rewind.c‎`

Lines changed: 2 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -645,6 +645,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`645`	`645`	`pg_free(histfile);`
`646`	`646`	`}`
`647`	`647`
	`648`	`+/* In debugging mode, print what we read */`
`648`	`649`	`if (debug)`
`649`	`650`	`{`
`650`	`651`	`inti;`
`@@ -656,10 +657,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`656`	`657`	`else`
`657`	`658`	`Assert(false);`
`658`	`659`
`659`		`-/*`
`660`		`- * Print the target timeline history.`
`661`		`- */`
`662`		`-for (i=0;i<targetNentries;i++)`
	`660`	`+for (i=0;i<*nentries;i++)`
`663`	`661`	`{`
`664`	`662`	`TimeLineHistoryEntry*entry;`
`665`	`663`

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!

Commitc943e2a

File tree

1 file changed

1 file changed

`‎src/bin/pg_rewind/pg_rewind.c‎`

0 commit comments