fix(export-backup): fix memory leak in backup export (#7452)

There was a memory leak because the program exits before DB close.

Author: NamanJain8

worker/export.go

@@ -607,7 +607,6 @@ func exportInternal(ctx context.Context, in *pb.ExportRequest, db *badger.DB,
 			return false
 		}
 
-		fmt.Println(pk)
 		// Do not pick keys storing parts of a multi-part list. They will be read
 		// from the main key.
 		if pk.HasStartUid {

worker/file_handler.go

@@ -363,7 +363,6 @@ func (h *fileHandler) ExportBackup(backupDir, exportDir, format string,
 				ch <- errors.Wrapf(err, "cannot open DB at %s", dir)
 				return
 			}
-			defer db.Close()
 
 			req := &pb.ExportRequest{
 				GroupId:     group,
@@ -375,6 +374,9 @@ func (h *fileHandler) ExportBackup(backupDir, exportDir, format string,
 			}
 
 			_, err = exportInternal(context.Background(), req, db, true)
+			// It is important to close the db before sending err to ch. Else, we will see a memory
+			// leak.
+			db.Close()
 			ch <- errors.Wrapf(err, "cannot export data inside DB at %s", dir)
 		}(gid)
 	}