fix linter

pingcap · Sep 5, 2022 · 9d02ffb · 9d02ffb
1 parent aca6609
commit 9d02ffb
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/ddl/lightning/BUILD.bazel b/ddl/lightning/BUILD.bazel
@@ -45,10 +45,14 @@ go_library(
 
 go_test(
     name = "lightning_test",
-    srcs = ["mem_root_test.go"],
+    srcs = [
+        "env_test.go",
+        "mem_root_test.go",
+    ],
     flaky = True,
     deps = [
         ":lightning",
+        "//config",
         "@com_github_stretchr_testify//require",
     ],
 )
diff --git a/ddl/lightning/engine_mgr.go b/ddl/lightning/engine_mgr.go
@@ -62,9 +62,6 @@ func (m *engineManager) Register(bc *BackendContext, job *model.Job, indexID int
 		id := openedEn.GetEngineUUID()
 		en = NewEngineInfo(bc.ctx, job.ID, indexID, cfg, openedEn, id, 1, m.MemRoot, m.DiskRoot)
 		m.Store(indexID, en)
-		if err != nil {
-			return errors.New(LitErrCreateEngineFail)
-		}
 		m.MemRoot.Consume(StructSizeEngineInfo)
 		m.MemRoot.ConsumeWithTag(encodeEngineTag(job.ID, indexID), engineCacheSize)
 	} else {

diff --git a/ddl/lightning/env.go b/ddl/lightning/env.go
@@ -114,5 +114,8 @@ func genLightningDataDir() (string, error) {
 	return sortPath, nil
 }
 
+// GenRLimitForTest is only used for test.
+var GenRLimitForTest = genRLimit
+
 // GenLightningDataDirForTest is only used for test.
 var GenLightningDataDirForTest = genLightningDataDir