Check pattern IsCompiled before Compile (#8486)

Tony-Romanov · web-flow · commit 6313a44e07f7 · 2024-09-02T12:28:36.000+02:00
diff --git a/ydb/core/kqp/compile_service/kqp_compile_computation_pattern_service.cpp b/ydb/core/kqp/compile_service/kqp_compile_computation_pattern_service.cpp
@@ -55,7 +55,7 @@ class TKqpCompileComputationPatternService : public TActorBootstrapped<TKqpCompi
         size_t patternsToCompileSize = PatternsToCompile.size();
         for (; PatternToCompileIndex < patternsToCompileSize && compilationIntervalMs > 0; ++PatternToCompileIndex) {
             auto& patternToCompile = PatternsToCompile[PatternToCompileIndex];
-            if (!patternToCompile.Entry->IsInCache.load()) {
+            if (!patternToCompile.Entry->IsInCache.load() || patternToCompile.Entry->Pattern->IsCompiled()) {
                 continue;
             }
 

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ class TKqpCompileComputationPatternService : public TActorBootstrapped<TKqpCompi`
`55`	`55`	`size_t patternsToCompileSize = PatternsToCompile.size();`
`56`	`56`	`for (; PatternToCompileIndex < patternsToCompileSize && compilationIntervalMs > 0; ++PatternToCompileIndex) {`
`57`	`57`	`auto& patternToCompile = PatternsToCompile[PatternToCompileIndex];`
`58`		`- if (!patternToCompile.Entry->IsInCache.load()) {`
	`58`	`+ if (!patternToCompile.Entry->IsInCache.load() \|\| patternToCompile.Entry->Pattern->IsCompiled()) {`
`59`	`59`	`continue;`
`60`	`60`	`}`
`61`	`61`