lightning: make OpLevelOptional suppress the error of DoChecksum

pingcap · Jul 24, 2023 · 39d8c16 · 39d8c16
1 parent bfc940a
commit 39d8c16
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/br/pkg/lightning/importer/table_import.go b/br/pkg/lightning/importer/table_import.go
@@ -1042,12 +1042,11 @@ func (tr *TableImporter) postProcess(
 				err = status.Error(codes.Unknown, "Checksum meets error.")
 			})
 			if err != nil {
-				if rc.cfg.PostRestore.Checksum == config.OpLevelOptional {
-					tr.logger.Warn("do checksum failed, will skip this error and go on", log.ShortError(err))
-					err = nil
+				if rc.cfg.PostRestore.Checksum != config.OpLevelOptional {
+					return false, err
 				}
-			} else {
-				return false, err
+				tr.logger.Warn("do checksum failed, will skip this error and go on", log.ShortError(err))
+				err = nil
 			}
 			if remoteChecksum != nil {
 				err = tr.compareChecksum(remoteChecksum, localChecksum)