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

Modified backpressure method for MergeTree [#METR-23488].

Browse Source
This commit is contained in:
Alexey Milovidov 2016-11-14 21:19:08 +03:00
parent 56e7e380ca
commit e7fccc4044
2 changed files with 1 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
dbms/include/DB/Storages/MergeTree/MergeTreeBlockOutputStream.h
View File

@ -5,11 +5,6 @@
#include <iomanip> #include <iomanip>
namespace ProfileEvents
{
extern const Event SynchronousMergeOnInsert;
}
namespace DB namespace DB
{ {
@ -21,17 +16,7 @@ public:
void writePrefix() override void writePrefix() override
{ {
/// If too much parts, do extra merges synchronously, in current thread. storage.data.delayInsertIfNeeded();
/// Why 10? - just in case, it is better than possibly infinite loop.
for (size_t i = 0; i < 10; ++i)
{
size_t parts_count = storage.data.getMaxPartsCountForMonth();
if (parts_count <= storage.data.settings.parts_to_delay_insert)
break;
ProfileEvents::increment(ProfileEvents::SynchronousMergeOnInsert);
storage.merge(0, {}, {}, {});
}
} }
void write(const Block & block) override void write(const Block & block) override

1
dbms/src/Common/ProfileEvents.cpp
View File

@ -46,7 +46,6 @@
M(DelayedInserts) \ M(DelayedInserts) \
M(RejectedInserts) \ M(RejectedInserts) \
M(DelayedInsertsMilliseconds) \ M(DelayedInsertsMilliseconds) \
M(SynchronousMergeOnInsert) \
\ \
M(ZooKeeperInit) \ M(ZooKeeperInit) \
M(ZooKeeperTransactions) \ M(ZooKeeperTransactions) \