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

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_r27025498
  
    --- Diff: 
tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/store/MemStore.java
 ---
    @@ -498,37 +498,16 @@ public void dropPartitionMethod(String databaseName, 
String tableName) throws Ca
       }
     
       @Override
    -  public void addPartitions(CatalogProtos.PartitionsProto 
partitionDescList) throws CatalogException {
    +  public List<CatalogProtos.PartitionDescProto> getPartitions(String 
databaseName, String tableName) throws CatalogException {
         throw new RuntimeException("not supported!");
    --- End diff --
    
    MemStore is used for tests, so we need to implement this function.


> 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