thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-21 09:10:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Finally fixed selection of destination of move TTL.

Browse Source
This commit is contained in:
Vladimir Chebotarev 2019-11-29 08:12:01 +03:00
parent 15fb64f54b
commit 8ed37801ca
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
dbms/src/Storages/MergeTree/MergeTreeData.cpp
View File

@ -3728,17 +3728,17 @@ const MergeTreeData::TTLEntry * MergeTreeData::selectMoveDestination(
{
const MergeTreeData::TTLEntry * result = nullptr;
/// Prefer TTL rule which went into action last.
time_t max_min_ttl = 0;
time_t max_max_ttl = 0;
for (const auto & ttl_entry : move_ttl_entries)
{
auto ttl_info_it = ttl_infos.moves_ttl.find(ttl_entry.result_column);
if (ttl_info_it != ttl_infos.moves_ttl.end()
&& ttl_info_it->second.min <= minimum_time
&& max_min_ttl <= ttl_info_it->second.min)
&& ttl_info_it->second.max <= minimum_time
&& max_max_ttl >= ttl_info_it->second.max)
{
result = &ttl_entry;
max_min_ttl = ttl_info_it->second.min;
max_max_ttl = ttl_info_it->second.max;
}
}