fix(PITR): compare TargetLSN with backup EndLSN instead of BeginLSN

mnencia · web-flow · commit 859c31d01b6b · 2025-01-07T16:09:54.000+01:00
Closes #6536

Signed-off-by: Marco Nenciarini &lt;marco.nenciarini@enterprisedb.com&gt;
diff --git a/pkg/catalog/catalog.go b/pkg/catalog/catalog.go
@@ -183,7 +183,7 @@ func (catalog *Catalog) findClosestBackupFromTargetLSN(
 		if (strconv.Itoa(barmanBackup.TimeLine) == targetTLI ||
 			// if targetTLI is not an integer, it will be ignored actually
 			currentTLIRegex.MatchString(targetTLI)) &&
-			types.LSN(barmanBackup.BeginLSN).Less(targetLSN) {
+			types.LSN(barmanBackup.EndLSN).Less(targetLSN) {
 			return &catalog.List[i], nil
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -183,7 +183,7 @@ func (catalog *Catalog) findClosestBackupFromTargetLSN(`
`183`	`183`	`if (strconv.Itoa(barmanBackup.TimeLine) == targetTLI \|\|`
`184`	`184`	`// if targetTLI is not an integer, it will be ignored actually`
`185`	`185`	`currentTLIRegex.MatchString(targetTLI)) &&`
`186`		`- types.LSN(barmanBackup.BeginLSN).Less(targetLSN) {`
	`186`	`+ types.LSN(barmanBackup.EndLSN).Less(targetLSN) {`
`187`	`187`	`return &catalog.List[i], nil`
`188`	`188`	`}`
`189`	`189`	`}`