Skip to content

Commit eb60ed0

Browse files
author
ilezhankin
committed
Disable Y_ENSURE() for improper invariant
Правильный фикс здесь: <https://github.com/ydb-platform/ydb/pull/11129/files> Ему неповезло попасть на переезд yql в аркадию, поэтому ждём его окончания. Y_ENSURE() вызывает падения в проде - до этого он был Y_ASSERT(), но так как проверяемый инвариант не всегда выполняется, то просто убрал. commit_hash:76966d1e5c858de4e099beccd34c9570dd9f6883
1 parent 205da07 commit eb60ed0

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

yql/essentials/minikql/computation/mkql_computation_pattern_cache.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,11 @@ class TComputationPatternLRUCache::TLRUPatternCacheImpl
7777

7878
const auto& entry = it->second.Entry;
7979

80-
Y_ENSURE(entry->Pattern->IsCompiled());
80+
// TODO(ilezhankin): wait until migration of yql to arcadia is complete and merge the proper fix from here:
81+
// https://github.com/ydb-platform/ydb/pull/11129
82+
if (!entry->Pattern->IsCompiled()) {
83+
return;
84+
}
8185

8286
if (it->second.LinkedInCompiledPatternLRUList()) {
8387
return;

0 commit comments

Comments
 (0)