NotificationsYou must be signed in to change notification settings
Fork4.9k
Star17.7k

Commit49dc191

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 parente56ccc8 commit49dc191Copy full SHA for 49dc191

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
`@@ -882,6 +882,7 @@ getTimelineHistory(TimeLineID tli, bool is_source, int *nentries)`
`882`	`882`	`pg_free(histfile);`
`883`	`883`	`}`
`884`	`884`
	`885`	`+/* In debugging mode, print what we read */`
`885`	`886`	`if (debug)`
`886`	`887`	`{`
`887`	`888`	`inti;`
`@@ -891,10 +892,7 @@ getTimelineHistory(TimeLineID tli, bool is_source, int *nentries)`
`891`	`892`	`else`
`892`	`893`	`pg_log_debug("Target timeline history:");`
`893`	`894`
`894`		`-/*`
`895`		`- * Print the target timeline history.`
`896`		`- */`
`897`		`-for (i=0;i<targetNentries;i++)`
	`895`	`+for (i=0;i<*nentries;i++)`
`898`	`896`	`{`
`899`	`897`	`TimeLineHistoryEntry*entry;`
`900`	`898`

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!

Commit49dc191

File tree

1 file changed

1 file changed

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

0 commit comments