Skip to content

Commit 9415ab6

Browse files
committed
restore ascii ignore case match (#19334)
Conflicts: .github/config/muted_ya.txt
1 parent 562e653 commit 9415ab6

File tree

4 files changed

+16
-18
lines changed

4 files changed

+16
-18
lines changed

.github/config/muted_ya.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@ ydb/core/kqp/ut/olap KqpOlapAggregations.BlockGenericWithDistinct
2424
ydb/core/kqp/ut/olap KqpOlapSysView.StatsSysViewBytesColumnActualization
2525
ydb/core/kqp/ut/olap KqpOlapSysView.StatsSysViewBytesDictActualization
2626
ydb/core/kqp/ut/olap KqpOlapSysView.StatsSysViewBytesDictStatActualization
27-
ydb/core/kqp/ut/olap KqpOlapJson.BloomNGrammIndexesVariants
28-
ydb/core/kqp/ut/olap KqpOlapJson.CompactionVariants
2927
ydb/core/kqp/ut/olap KqpOlapJson.DuplicationCompactionVariants
3028
ydb/core/kqp/ut/olap KqpOlapWrite.TierDraftsGCWithRestart
3129
ydb/core/kqp/ut/olap [*/*] chunk chunk

ydb/core/formats/arrow/program/kernel_logic.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ class TLogicMatchString: public IKernelLogic {
151151
class TLogicMatchAsciiEqualsIgnoreCase: public TLogicMatchString {
152152
private:
153153
static TString GetClassNameStatic() {
154-
return "String.AsciiEqualsIgnoreCase";
154+
return "String._yql_AsciiEqualsIgnoreCase";
155155
}
156156
public:
157157
TLogicMatchAsciiEqualsIgnoreCase() : TLogicMatchString(TIndexCheckOperation::EOperation::Contains, false, false) {
@@ -163,7 +163,7 @@ class TLogicMatchAsciiEqualsIgnoreCase: public TLogicMatchString {
163163
class TLogicMatchAsciiContainsIgnoreCase: public TLogicMatchString {
164164
private:
165165
static TString GetClassNameStatic() {
166-
return "String.AsciiContainsIgnoreCase";
166+
return "String._yql_AsciiContainsIgnoreCase";
167167
}
168168
public:
169169
TLogicMatchAsciiContainsIgnoreCase() : TLogicMatchString(TIndexCheckOperation::EOperation::Contains, false, false) {
@@ -175,7 +175,7 @@ class TLogicMatchAsciiContainsIgnoreCase: public TLogicMatchString {
175175
class TLogicMatchAsciiStartsWithIgnoreCase: public TLogicMatchString {
176176
private:
177177
static TString GetClassNameStatic() {
178-
return "String.AsciiStartsWithIgnoreCase";
178+
return "String._yql_AsciiStartsWithIgnoreCase";
179179
}
180180
public:
181181
TLogicMatchAsciiStartsWithIgnoreCase() : TLogicMatchString(TIndexCheckOperation::EOperation::StartsWith, false, false) {
@@ -187,7 +187,7 @@ class TLogicMatchAsciiStartsWithIgnoreCase: public TLogicMatchString {
187187
class TLogicMatchAsciiEndsWithIgnoreCase: public TLogicMatchString {
188188
private:
189189
static TString GetClassNameStatic() {
190-
return "String.AsciiEndsWithIgnoreCase";
190+
return "String._yql_AsciiEndsWithIgnoreCase";
191191
}
192192
public:
193193
TLogicMatchAsciiEndsWithIgnoreCase() : TLogicMatchString(TIndexCheckOperation::EOperation::EndsWith, false, false) {

ydb/core/kqp/opt/physical/predicate_collector.cpp

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,10 @@ using namespace NYql;
1010
using namespace NYql::NNodes;
1111

1212
THashMap<TString, TString> IgnoreCaseSubstringMatchFunctions = {
13-
{"EqualsIgnoreCase", "String.AsciiEqualsIgnoreCase"},
14-
{"StartsWithIgnoreCase", "String.AsciiStartsWithIgnoreCase"},
15-
{"EndsWithIgnoreCase", "String.AsciiEndsWithIgnoreCase"},
16-
{"StringContainsIgnoreCase", "String.AsciiContainsIgnoreCase"}
13+
{"EqualsIgnoreCase", "String._yql_AsciiEqualsIgnoreCase"},
14+
{"StartsWithIgnoreCase", "String._yql_AsciiStartsWithIgnoreCase"},
15+
{"EndsWithIgnoreCase", "String._yql_AsciiEndsWithIgnoreCase"},
16+
{"StringContainsIgnoreCase", "String._yql_AsciiContainsIgnoreCase"}
1717
};
1818

1919
namespace {
@@ -124,14 +124,14 @@ bool CanPushdownStringUdf(const TExprNode& udf, bool pushdownSubstring) {
124124
}
125125
const auto& name = udf.Head().Content();
126126
static const THashSet<TString> substringMatchUdfs = {
127-
"String.AsciiEqualsIgnoreCase",
127+
"String._yql_AsciiEqualsIgnoreCase",
128128

129129
"String.Contains",
130-
"String.AsciiContainsIgnoreCase",
130+
"String._yql_AsciiContainsIgnoreCase",
131131
"String.StartsWith",
132-
"String.AsciiStartsWithIgnoreCase",
132+
"String._yql_AsciiStartsWithIgnoreCase",
133133
"String.EndsWith",
134-
"String.AsciiEndsWithIgnoreCase"
134+
"String._yql_AsciiEndsWithIgnoreCase"
135135
};
136136
return substringMatchUdfs.contains(name);
137137
}

ydb/core/kqp/ut/olap/kqp_olap_ut.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1240,13 +1240,13 @@ Y_UNIT_TEST_SUITE(KqpOlap) {
12401240
R"(`uid` LIKE "_id%000_")",
12411241
R"(`uid` ILIKE "UID%002")",
12421242

1243-
//R"(Udf(String::AsciiEqualsIgnoreCase)(`uid`, "UI"))", //bug #19270
1243+
R"(Udf(String::_yql_AsciiEqualsIgnoreCase)(`uid`, "UI"))",
12441244
R"(Udf(String::Contains)(`uid`, "UI"))",
1245-
//R"(Udf(String::AsciiContainsIgnoreCase)(`uid`, "UI"))", //bug #19270
1245+
R"(Udf(String::_yql_AsciiContainsIgnoreCase)(`uid`, "UI"))",
12461246
R"(Udf(String::StartsWith)(`uid`, "UI"))",
1247-
//R"(Udf(String::AsciiStartsWithIgnoreCase)(`uid`, "UI"))", //bug #19270
1247+
R"(Udf(String::_yql_AsciiStartsWithIgnoreCase)(`uid`, "UI"))",
12481248
R"(Udf(String::EndsWith)(`uid`, "UI"))",
1249-
//R"(Udf(String::AsciiEndsWithIgnoreCase)(`uid`, "UI"))", //bug #19270
1249+
R"(Udf(String::_yql_AsciiEndsWithIgnoreCase)(`uid`, "UI"))",
12501250
};
12511251

12521252
for (const auto& predicate: testData) {

0 commit comments

Comments
 (0)