Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
MrLolthe1st committed Aug 15, 2024
1 parent 510663f commit 2a66eaa
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
6 changes: 0 additions & 6 deletions ydb/library/yql/core/ut/yql_column_order_ut.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,6 @@ using namespace NYql;

Y_UNIT_TEST_SUITE(TYqlColumnOrder) {

Y_UNIT_TEST(ColumnOrderIgnoreCase) {
TColumnOrder order;
UNIT_ASSERT_EQUAL(order.AddColumn("a"), "a");
UNIT_ASSERT_EQUAL(order.AddColumn("A"), "A_generated_2");
}

Y_UNIT_TEST(ColumnOrderShrink) {
TColumnOrder order;
UNIT_ASSERT_EQUAL(order.AddColumn("a"), "a");
Expand Down
4 changes: 3 additions & 1 deletion ydb/library/yql/core/yql_type_annotation.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,7 @@ TString TColumnOrder::Find(const TString& name) const {
TColumnOrder& TColumnOrder::operator=(const TColumnOrder& rhs) {
GeneratedToOriginal_ = rhs.GeneratedToOriginal_;
Order_ = rhs.Order_;
UseCountLcase_ = rhs.UseCountLcase_;
UseCount_ = rhs.UseCount_;
return *this;
}
Expand Down Expand Up @@ -152,7 +153,7 @@ TString TColumnOrder::AddColumn(const TString& name) {

bool TColumnOrder::IsDuplicatedIgnoreCase(const TString& name) const {
auto it = UseCountLcase_.find(to_lower(name));
return it != UseCount_.end() && it->second > 1;
return it != UseCountLcase_.end() && it->second > 1;
}

void TColumnOrder::Shrink(size_t remain) {
Expand All @@ -179,6 +180,7 @@ void TColumnOrder::Clear() {
Order_.clear();
GeneratedToOriginal_.clear();
UseCount_.clear();
UseCountLcase_.clear();
}

void TColumnOrder::EraseIf(const std::function<bool(const TString&)>& fn) {
Expand Down

0 comments on commit 2a66eaa

Please sign in to comment.