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

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

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

    https://github.com/apache/tajo/pull/448#discussion_r27023351
  
    --- Diff: 
tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogConstants.java
 ---
    @@ -48,11 +48,16 @@
       public static final String TB_STATISTICS = "STATS";
       public static final String TB_PARTITION_METHODS = "PARTITION_METHODS";
       public static final String TB_PARTTIONS = "PARTITIONS";
    +  public static final String TB_PARTTION_KEYS = "PARTITION_KEYS";
     
       public static final String COL_TABLESPACE_PK = "SPACE_ID";
       public static final String COL_DATABASES_PK = "DB_ID";
       public static final String COL_TABLES_PK = "TID";
       public static final String COL_TABLES_NAME = "TABLE_NAME";
    +
    +  public static final String COL_PARTITIONS_PK = "PID";
    --- End diff --
    
    I think that it would be better if the name is more intuitive.


> Add alter partition method to CatalogStore
> ------------------------------------------
>
>                 Key: TAJO-1284
>                 URL: https://issues.apache.org/jira/browse/TAJO-1284
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: catalog
>            Reporter: Jaehwa Jung
>            Assignee: Jaehwa Jung
>         Attachments: TAJO-1260.png, TAJO-1260.sql, TAJO-1284.patch, 
> TAJO-1284_2.patch, TAJO-1284_2.png, TAJO-1284_2.sql
>
>
> This patch will implement  ALTER TABLE ADD PARTITION method and ALTER TABLE 
> DROP PARTITION method to CatalogStore.



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

Reply via email to