thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-03 13:02:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Make use of ALTER_MOVE_PARTITION privilege.

Browse Source
This commit is contained in:
pufit 2024-11-08 01:47:11 -05:00
parent a15e5b72af
commit cf86df9966
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/Access/ContextAccess.cpp
View File

@ -168,6 +168,12 @@ AccessRights ContextAccess::addImplicitAccessRights(const AccessRights & access,
res |= show_databases;
}
static const AccessFlags alter_delete = AccessType::ALTER_DELETE;
static const AccessFlags select = AccessType::SELECT;
static const AccessFlags move_partition = AccessType::ALTER_MOVE_PARTITION;
if ((res & alter_delete) && (res & select) && level <= 2)
res |= move_partition;
max_flags |= res;
return res;

4
src/Interpreters/InterpreterAlterQuery.cpp
View File

@ -467,11 +467,11 @@ AccessRightsElements InterpreterAlterQuery::getRequiredAccessForCommand(const AS
required_access.emplace_back(AccessType::ALTER_MOVE_PARTITION, database, table);
break;
case DataDestinationType::TABLE:
required_access.emplace_back(AccessType::SELECT | AccessType::ALTER_DELETE, database, table);
required_access.emplace_back(AccessType::ALTER_MOVE_PARTITION, database, table);
required_access.emplace_back(AccessType::INSERT, command.to_database, command.to_table);
break;
case DataDestinationType::SHARD:
required_access.emplace_back(AccessType::SELECT | AccessType::ALTER_DELETE, database, table);
required_access.emplace_back(AccessType::ALTER_MOVE_PARTITION, database, table);
required_access.emplace_back(AccessType::MOVE_PARTITION_BETWEEN_SHARDS);
break;
case DataDestinationType::DELETE: