Skip to content

Commit 982a43b

Browse files
authored
Merge pull request ClickHouse#80622 from ClickHouse/ahmadov/gin-cosmetics
cosmetics: simplify gin condition check
2 parents 7f8c9ec + a59a8cb commit 982a43b

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/Storages/MergeTree/MergeTreeDataSelectExecutor.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1733,11 +1733,10 @@ MarkRanges MergeTreeDataSelectExecutor::filterMarksUsingIndex(
17331733
else
17341734
{
17351735
bool result = false;
1736-
const auto * gin_filter_condition = dynamic_cast<const MergeTreeIndexConditionGin *>(&*condition);
1737-
if (!gin_filter_condition)
1738-
result = condition->mayBeTrueOnGranule(granule);
1739-
else
1736+
if (const auto * gin_filter_condition = dynamic_cast<const MergeTreeIndexConditionGin *>(&*condition))
17401737
result = cache_in_store.store ? gin_filter_condition->mayBeTrueOnGranuleInPart(granule, cache_in_store) : true;
1738+
else
1739+
result = condition->mayBeTrueOnGranule(granule);
17411740

17421741
if (!result)
17431742
continue;

0 commit comments

Comments
 (0)