some adjustments

Author: arthurpassos

src/Access/Common/AccessType.h

@@ -74,6 +74,7 @@ enum class AccessType : uint8_t
                                                                        enabled implicitly by the grant ALTER_TABLE */\
     M(ALTER_SETTINGS, "ALTER SETTING, ALTER MODIFY SETTING, MODIFY SETTING, RESET SETTING", TABLE, ALTER_TABLE) /* allows to execute ALTER MODIFY SETTING */\
     M(ALTER_MOVE_PARTITION, "ALTER MOVE PART, MOVE PARTITION, MOVE PART", TABLE, ALTER_TABLE) \
+    M(ALTER_EXPORT_PARTITION, "ALTER EXPORT PART, EXPORT PARTITION, EXPORT PART", TABLE, ALTER_TABLE) \
     M(ALTER_FETCH_PARTITION, "ALTER FETCH PART, FETCH PARTITION", TABLE, ALTER_TABLE) \
     M(ALTER_FREEZE_PARTITION, "FREEZE PARTITION, UNFREEZE", TABLE, ALTER_TABLE) \
     M(ALTER_UNLOCK_SNAPSHOT, "UNLOCK SNAPSHOT", TABLE, ALTER_TABLE) \

src/Interpreters/InterpreterAlterQuery.cpp

@@ -502,9 +502,9 @@ AccessRightsElements InterpreterAlterQuery::getRequiredAccessForCommand(const AS
             required_access.emplace_back(AccessType::ALTER_DELETE | AccessType::INSERT, database, table);
             break;
         }
-        case ASTAlterCommand::EXPORT_PART:
+        case ASTAlterCommand::EXPORT_PARTITION:
         {
-            required_access.emplace_back(AccessType::ALTER_MOVE_PARTITION, database, table);
+            required_access.emplace_back(AccessType::ALTER_EXPORT_PARTITION, database, table);
             required_access.emplace_back(AccessType::INSERT, command.to_database, command.to_table);
             break;
         }

src/Parsers/ASTAlterQuery.cpp

@@ -355,7 +355,7 @@ void ASTAlterCommand::formatImpl(WriteBuffer & ostr, const FormatSettings & sett
             ostr << quoteString(move_destination_name);
         }
     }
-    else if (type == ASTAlterCommand::EXPORT_PART)
+    else if (type == ASTAlterCommand::EXPORT_PARTITION)
     {
         ostr << (settings.hilite ? hilite_keyword : "") << "EXPORT " << (part ? "PART " : "PARTITION ")
              << (settings.hilite ? hilite_none : "");

src/Parsers/ASTAlterQuery.h

@@ -71,7 +71,7 @@ class ASTAlterCommand : public IAST
         FREEZE_ALL,
         UNFREEZE_PARTITION,
         UNFREEZE_ALL,
-        EXPORT_PART,
+        EXPORT_PARTITION,
 
         DELETE,
         UPDATE,

src/Parsers/CommonParsers.h

@@ -331,6 +331,7 @@ namespace DB
     MR_MACROS(MONTHS, "MONTHS") \
     MR_MACROS(MOVE_PART, "MOVE PART") \
     MR_MACROS(MOVE_PARTITION, "MOVE PARTITION") \
+    MR_MACROS(EXPORT_PART, "EXPORT PART") \
     MR_MACROS(EXPORT_PARTITION, "EXPORT PARTITION") \
     MR_MACROS(MOVE, "MOVE") \
     MR_MACROS(MS, "MS") \

src/Parsers/ParserAlterQuery.cpp

@@ -82,7 +82,8 @@ bool ParserAlterCommand::parseImpl(Pos & pos, ASTPtr & node, Expected & expected
     ParserKeyword s_forget_partition(Keyword::FORGET_PARTITION);
     ParserKeyword s_move_partition(Keyword::MOVE_PARTITION);
     ParserKeyword s_move_part(Keyword::MOVE_PART);
-    ParserKeyword s_export_part(Keyword::EXPORT_PARTITION);
+    ParserKeyword s_export_part(Keyword::EXPORT_PART);
+    ParserKeyword s_export_partition(Keyword::EXPORT_PARTITION);
     ParserKeyword s_drop_detached_partition(Keyword::DROP_DETACHED_PARTITION);
     ParserKeyword s_drop_detached_part(Keyword::DROP_DETACHED_PART);
     ParserKeyword s_fetch_partition(Keyword::FETCH_PARTITION);
@@ -565,13 +566,29 @@ bool ParserAlterCommand::parseImpl(Pos & pos, ASTPtr & node, Expected & expected
                     command->move_destination_name = ast_space_name->as<ASTLiteral &>().value.safeGet<String>();
                 }
             }
+            else if (s_export_partition.ignore(pos, expected))
+            {
+                if (!parser_partition.parse(pos, command_partition, expected))
+                    return false;
+
+                command->type = ASTAlterCommand::EXPORT_PARTITION;
+
+                if (!s_to_table.ignore(pos, expected))
+                {
+                    return false;
+                }
+
+                if (!parseDatabaseAndTableName(pos, expected, command->to_database, command->to_table))
+                    return false;
+                command->move_destination_type = DataDestinationType::TABLE;
+            }
             else if (s_export_part.ignore(pos, expected))
             {
                 if (!parser_partition.parse(pos, command_partition, expected))
                     return false;
 
-                command->type = ASTAlterCommand::EXPORT_PART;
-                //                command->part = true;
+                command->type = ASTAlterCommand::EXPORT_PARTITION;
+                command->part = true;
 
                 if (!s_to_table.ignore(pos, expected))
                 {

src/Storages/MergeTree/MergeTreeData.cpp

@@ -5940,7 +5940,7 @@ Pipe MergeTreeData::alterPartition(
                 }
             }
             break;
-            case PartitionCommand::EXPORT_PART:
+            case PartitionCommand::EXPORT_PARTITION:
             {
                 exportPartitionToTable(command, query_context);
                 break;

src/Storages/PartitionCommands.cpp

@@ -52,10 +52,10 @@ std::optional<PartitionCommand> PartitionCommand::parse(const ASTAlterCommand *
         res.part = command_ast->part;
         return res;
     }
-    if (command_ast->type == ASTAlterCommand::EXPORT_PART)
+    if (command_ast->type == ASTAlterCommand::EXPORT_PARTITION)
     {
         PartitionCommand res;
-        res.type = EXPORT_PART;
+        res.type = EXPORT_PARTITION;
         res.partition = command_ast->partition->clone();
         res.part = command_ast->part;
         res.move_destination_type = PartitionCommand::MoveDestinationType::TABLE;

src/Storages/PartitionCommands.h

@@ -33,7 +33,7 @@ struct PartitionCommand
         UNFREEZE_ALL_PARTITIONS,
         UNFREEZE_PARTITION,
         REPLACE_PARTITION,
-        EXPORT_PART,
+        EXPORT_PARTITION,
     };
 
     Type type = UNKNOWN;