diff options
| author | ilezhankin <ilezhankin@yandex-team.com> | 2024-11-22 17:43:41 +0300 |
|---|---|---|
| committer | ilezhankin <ilezhankin@yandex-team.com> | 2024-11-22 17:56:56 +0300 |
| commit | eb60ed0bccfea7b15375045ffd118e90af469491 (patch) | |
| tree | f597c57e630c08792c1c730edc6292ab48aadbcd /yql/essentials/minikql | |
| parent | 205da070631b2e9e41b8b430c6a5ffb3869b1501 (diff) | |
| download | ydb-eb60ed0bccfea7b15375045ffd118e90af469491.tar.gz | |
Disable Y_ENSURE() for improper invariant
Правильный фикс здесь: <https://github.com/ydb-platform/ydb/pull/11129/files>
Ему неповезло попасть на переезд yql в аркадию, поэтому ждём его окончания.
Y_ENSURE() вызывает падения в проде - до этого он был Y_ASSERT(), но так как проверяемый инвариант не всегда выполняется, то просто убрал.
commit_hash:76966d1e5c858de4e099beccd34c9570dd9f6883
Diffstat (limited to 'yql/essentials/minikql')
| -rw-r--r-- | yql/essentials/minikql/computation/mkql_computation_pattern_cache.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/yql/essentials/minikql/computation/mkql_computation_pattern_cache.cpp b/yql/essentials/minikql/computation/mkql_computation_pattern_cache.cpp index 8a4c9e70ce..57164430cc 100644 --- a/yql/essentials/minikql/computation/mkql_computation_pattern_cache.cpp +++ b/yql/essentials/minikql/computation/mkql_computation_pattern_cache.cpp @@ -77,7 +77,11 @@ public: const auto& entry = it->second.Entry; - Y_ENSURE(entry->Pattern->IsCompiled()); + // TODO(ilezhankin): wait until migration of yql to arcadia is complete and merge the proper fix from here: + // https://github.com/ydb-platform/ydb/pull/11129 + if (!entry->Pattern->IsCompiled()) { + return; + } if (it->second.LinkedInCompiledPatternLRUList()) { return; |