Fix sorting in detailed mongodb backup list

citusdata · Sep 14, 2022 · 98f1b0d · 98f1b0d
1 parent 71fb7e9
commit 98f1b0d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/internal/databases/mongo/backup_list_handler.go b/internal/databases/mongo/backup_list_handler.go
@@ -45,7 +45,7 @@ func HandleDetailedBackupList(folder storage.Folder, output io.Writer, pretty, j
 	}
 
 	sort.Slice(backupDetails, func(i, j int) bool {
-		return backupDetails[i].FinishLocalTime.After(backupDetails[j].FinishLocalTime)
+		return backupDetails[i].FinishLocalTime.Before(backupDetails[j].FinishLocalTime)
 	})
 
 	switch {

diff --git a/internal/databases/mongo/binary/backup.go b/internal/databases/mongo/binary/backup.go
@@ -62,7 +62,9 @@ func (backupService *BackupService) DoBackup(backupName string, permanent bool)
 	defer func() {
 		if err != nil {
 			closeErr := backupCursor.Close(backupService.Context)
-			tracelog.ErrorLogger.Printf("Unable to close backup cursor: %+v", closeErr)
+			if closeErr != nil {
+				tracelog.ErrorLogger.Printf("Unable to close backup cursor: %+v", closeErr)
+			}
 		}
 	}()