Skip to content

Commit 2ba1427

Browse files
fix tests canonization and case withno CS object (#3277)
1 parent f94bbf3 commit 2ba1427

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

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

+2-2
Original file line numberDiff line numberDiff line change
@@ -1675,7 +1675,7 @@ Y_UNIT_TEST_SUITE(KqpOlap) {
16751675
["some prefix xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"];
16761676
["5"];
16771677
1000005u;
1678-
["uid_1000005"]
1678+
"uid_1000005"
16791679
]])");
16801680
}
16811681
};
@@ -6079,7 +6079,7 @@ Y_UNIT_TEST_SUITE(KqpOlap) {
60796079
UNIT_ASSERT_C(result.IsSuccess(), result.GetIssues().ToString());
60806080
TString output = FormatResultSetYson(result.GetResultSet(0));
60816081
Cout << output << Endl;
6082-
CompareYson(output, R"([[1000001u;["1"];["uid_1000001"];[1]]])");
6082+
CompareYson(output, R"([[1000001u;["1"];"uid_1000001";[1]]])");
60836083
}
60846084

60856085
Y_UNIT_TEST(OlapRead_GenericQuery) {

ydb/core/tx/columnshard/engines/changes/with_appended.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ void TChangesWithAppend::DoWriteIndexOnExecute(NColumnShard::TColumnShard* self,
1717
}
1818
const auto predRemoveDroppedTable = [self](const TPortionInfoWithBlobs& item) {
1919
auto& portionInfo = item.GetPortionInfo();
20-
if (!self->TablesManager.HasTable(portionInfo.GetPathId()) || self->TablesManager.GetTable(portionInfo.GetPathId()).IsDropped()) {
20+
if (!!self && (!self->TablesManager.HasTable(portionInfo.GetPathId()) || self->TablesManager.GetTable(portionInfo.GetPathId()).IsDropped())) {
2121
AFL_WARN(NKikimrServices::TX_COLUMNSHARD)("event", "skip_inserted_data")("reason", "table_removed")("path_id", portionInfo.GetPathId());
2222
return true;
2323
} else {

0 commit comments

Comments
 (0)