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

cancel merges before acquiring lock for truncate

Browse Source
This commit is contained in:
Alexander Tokmakov 2022-02-03 23:44:53 +03:00
parent 9b844c6b42
commit 3e700e854d
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/Interpreters/InterpreterDropQuery.cpp
View File

@ -35,6 +35,10 @@ namespace ErrorCodes
extern const int TABLE_IS_READ_ONLY;
}
namespace ActionLocks
{
extern const StorageActionBlockType PartsMerge;
}
static DatabasePtr tryGetDatabase(const String & database_name, bool if_exists)
{
@ -202,7 +206,15 @@ BlockIO InterpreterDropQuery::executeToTableImpl(ContextPtr context_, ASTDropQue
table->checkTableCanBeDropped();
TableExclusiveLockHolder table_lock;
/// We don't need this lock for ReplicatedMergeTree
if (!table->supportsReplication())
{
/// And for simple MergeTree we can stop merges before acquiring the lock
auto merges_blocker = table->getActionLock(ActionLocks::PartsMerge);
auto table_lock = table->lockExclusively(context_->getCurrentQueryId(), context_->getSettingsRef().lock_acquire_timeout);
}
auto metadata_snapshot = table->getInMemoryMetadataPtr();
/// Drop table data, don't touch metadata
table->truncate(query_ptr, metadata_snapshot, context_, table_lock);