Fixed the bug that caused GC Verify to panic when running during checkpoint but not completed. (#22434)

Fixed the bug that caused GC Verify to panic when running during checkpoint but not completed.

Approved by: @XuPeng-SH

Author: LeftHandCold

pkg/vm/engine/tae/db/gc/v3/check.go

@@ -203,14 +203,17 @@ func (c *gcChecker) Verify(ctx context.Context, mp *mpool.MPool) (returnStr stri
 
 	// Collect all checkpoint files
 	var ckpObjectCount int
-	ckps := c.cleaner.checkpointCli.GetAllGlobalCheckpoints()
-	ckps = append(ckps, c.cleaner.checkpointCli.GetAllIncrementalCheckpoints()...)
+	checkpoints := c.cleaner.checkpointCli.GetAllGlobalCheckpoints()
+	checkpoints = append(checkpoints, c.cleaner.checkpointCli.GetAllIncrementalCheckpoints()...)
 
 	compacted := c.cleaner.checkpointCli.GetCompacted()
 	if compacted != nil {
-		ckps = append(ckps, compacted)
+		checkpoints = append(checkpoints, compacted)
 	}
-	for _, ckp := range ckps {
+	for _, ckp := range checkpoints {
+		if !ckp.IsFinished() {
+			continue
+		}
 		var files []string
 		files, err = getCheckpointLocation(ctx, ckp, c.cleaner.fs)
 		if err != nil {

pkg/vm/engine/tae/db/gc/v3/window.go

@@ -672,7 +672,6 @@ func (w *GCWindow) Details(ctx context.Context, snapshotMeta *logtail.SnapshotMe
 			for tid, entry := range tables {
 				accountID, ok := snapshotMeta.GetAccountId(tid)
 				if !ok {
-					logutil.Error("GetAccountId is error")
 					continue
 				}
 				if details[accountID] == nil {
@@ -702,7 +701,6 @@ func (w *GCWindow) Details(ctx context.Context, snapshotMeta *logtail.SnapshotMe
 		for tid, entry := range tables {
 			accountID, ok := snapshotMeta.GetAccountId(tid)
 			if !ok {
-				logutil.Error("GetAccountId is error")
 				continue
 			}
 			if details[accountID] == nil {