NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitb5a5027

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 parent2de129b commitb5a5027Copy full SHA for b5a5027

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
`@@ -837,6 +837,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`837`	`837`	`pg_free(histfile);`
`838`	`838`	`}`
`839`	`839`
	`840`	`+/* In debugging mode, print what we read */`
`840`	`841`	`if (debug)`
`841`	`842`	`{`
`842`	`843`	`inti;`
`@@ -848,10 +849,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`848`	`849`	`else`
`849`	`850`	`Assert(false);`
`850`	`851`
`851`		`-/*`
`852`		`- * Print the target timeline history.`
`853`		`- */`
`854`		`-for (i=0;i<targetNentries;i++)`
	`852`	`+for (i=0;i<*nentries;i++)`
`855`	`853`	`{`
`856`	`854`	`TimeLineHistoryEntry*entry;`
`857`	`855`

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!

Commitb5a5027

File tree

1 file changed

1 file changed

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

0 commit comments