Skip to content

Commit

Permalink
Merge some fixes (#6490)
Browse files Browse the repository at this point in the history
Co-authored-by: Tony-Romanov <150126326+Tony-Romanov@users.noreply.github.com>
  • Loading branch information
gridnevvvit and Tony-Romanov authored Jul 10, 2024
1 parent 7c2c3cd commit 0711d51
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions ydb/core/kqp/query_compiler/kqp_olap_compiler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -856,6 +856,8 @@ TTypedColumn GetOrCreateColumnIdAndType(const TExprBase& node, TKqpOlapCompileCo
return BuildLogicalNot(maybeNot.Cast().Value(), ctx);
} else if (const auto& maybeJsonValue = node.Maybe<TKqpOlapJsonValue>()) {
return ConvertJsonValueToColumn(maybeJsonValue.Cast(), ctx);
} else if (const auto& maybeJsonValue = node.Maybe<TKqpOlapJsonExists>()) {
return CompileJsonExists(maybeJsonValue.Cast(), ctx);
} else if (const auto& maybeApply = node.Maybe<TKqpOlapApply>()) {
return CompileYqlKernelScalarApply(maybeApply.Cast(), ctx);
}
Expand Down
2 changes: 1 addition & 1 deletion ydb/core/kqp/ut/olap/kqp_olap_ut.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -939,7 +939,6 @@ Y_UNIT_TEST_SUITE(KqpOlap) {
R"(`resource_id` = "10001")",
R"(`resource_id` != "10001")",
R"("XXX" == "YYY" OR `resource_id` != "10001")",
R"(`resource_id` != "10001" XOR "XXX" == "YYY")",
R"(`level` = 1)",
R"(`level` = Int8("1"))",
R"(`level` = Int16("1"))",
Expand Down Expand Up @@ -1021,6 +1020,7 @@ Y_UNIT_TEST_SUITE(KqpOlap) {
R"((`timestamp`, `level`) >= (Timestamp("1970-01-01T00:00:03.000001Z"), 3))",
#endif
#if SSA_RUNTIME_VERSION >= 5U
R"(`resource_id` != "10001" XOR "XXX" == "YYY")",
R"(IF(`level` > 3, -`level`, +`level`) < 2)",
R"(StartsWith(`message` ?? `resource_id`, "10000"))",
R"(NOT EndsWith(`message` ?? `resource_id`, "xxx"))",
Expand Down

0 comments on commit 0711d51

Please sign in to comment.