thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-23 16:12:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #55353 from ClickHouse/fix-garbage-2

Browse Source 
Fix trash optimization (up to a certain extent)
This commit is contained in:
Alexey Milovidov 2023-10-09 02:42:56 +02:00 committed by GitHub
commit b619d07baf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 37 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
src/Interpreters/LogicalExpressionsOptimizer.cpp
View File

@ -204,26 +204,41 @@ inline ASTs & getFunctionOperands(const ASTFunction * or_function)
bool LogicalExpressionsOptimizer::isLowCardinalityEqualityChain(const std::vector<ASTFunction *> & functions) const
{
if (functions.size() > 1)
if (functions.size() <= 1)
return false;
if (!functions[0])
return false;
/// Check if the identifier has LowCardinality type.
auto & first_operands = getFunctionOperands(functions.at(0));
if (first_operands.empty())
return false;
if (!first_operands[0])
return false;
const auto * identifier = first_operands.at(0)->as<ASTIdentifier>();
if (!identifier)
return false;
auto pos = IdentifierSemantic::getMembership(*identifier);
if (!pos)
pos = IdentifierSemantic::chooseTableColumnMatch(*identifier, tables_with_columns, true);
if (!pos)
return false;
if (*pos >= tables_with_columns.size())
return false;
if (auto data_type_and_name = tables_with_columns.at(*pos).columns.tryGetByName(identifier->shortName()))
{
/// Check if identifier is LowCardinality type
auto & first_operands = getFunctionOperands(functions[0]);
const auto * identifier = first_operands[0]->as<ASTIdentifier>();
if (identifier)
{
auto pos = IdentifierSemantic::getMembership(*identifier);
if (!pos)
pos = IdentifierSemantic::chooseTableColumnMatch(*identifier, tables_with_columns, true);
if (pos)
{
if (auto data_type_and_name = tables_with_columns[*pos].columns.tryGetByName(identifier->shortName()))
{
if (typeid_cast<const DataTypeLowCardinality *>(data_type_and_name->type.get()))
return true;
}
}
}
if (typeid_cast<const DataTypeLowCardinality *>(data_type_and_name->type.get()))
return true;
}
return false;
}

1
tests/queries/0_stateless/02893_trash_optimization.reference Normal file
View File

@ -0,0 +1 @@
0

3
tests/queries/0_stateless/02893_trash_optimization.sql Normal file
View File

@ -0,0 +1,3 @@
SELECT *
FROM merge('system', '^one$') AS one
WHERE (one.dummy = 0) OR (one.dummy = 1);