[ 
https://issues.apache.org/jira/browse/TAJO-1673?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14902953#comment-14902953
 ] 

ASF GitHub Bot commented on TAJO-1673:
--------------------------------------

Github user jihoonson commented on a diff in the pull request:

    https://github.com/apache/tajo/pull/626#discussion_r40111028
  
    --- Diff: 
tajo-core/src/main/java/org/apache/tajo/master/exec/DDLExecutor.java ---
    @@ -529,24 +536,137 @@ public void alterTable(TajoMaster.MasterContext 
context, final QueryContext quer
             catalog.alterTable(CatalogUtil.addOrDropPartition(qualifiedName, 
alterTable.getPartitionColumns(),
                 alterTable.getPartitionValues(), alterTable.getLocation(), 
AlterTableType.DROP_PARTITION));
     
    -        // When dropping partition on an managed table, the data will be 
delete from file system.
    -        if (!desc.isExternal()) {
    +        // When dropping partition on a table, the data in the table will 
NOT be deleted from the file system.
    +        // But if PURGE is specified, the partition data will be deleted.
    --- End diff --
    
    How about changing the comment to as follows?
    ```When dropping a partition on a table, its data will NOT be deleted if 
the 'PURGE' option is not specified.```


> Implement recover partitions
> ----------------------------
>
>                 Key: TAJO-1673
>                 URL: https://issues.apache.org/jira/browse/TAJO-1673
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: Catalog
>            Reporter: Jaehwa Jung
>            Assignee: Jaehwa Jung
>             Fix For: 0.11.0, 0.12.0
>
>         Attachments: TAJO-1673.patch, TAJO-1673_2.patch
>
>
> Tajo stores a list of partitions for each table in its CatalogStore. If, 
> however, new partitions are directly added to HDFS, the CatalogStore will not 
> be aware of these partitions unless the user runs ALTER TABLE table_name ADD 
> PARTITION commands on each of the newly added partitions.
> However, users can run a CatalogStore check command with the repair table 
> option:
> {code:xml}
> MSCK REPAIR TABLE <table_name>
> {code}
> For the reference, I've referenced hive msck command.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to