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

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

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

    https://github.com/apache/tajo/pull/1036#discussion_r70083050
  
    --- Diff: 
tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTable.java ---
    @@ -43,6 +44,8 @@
       private AlterTableOpType alterTableOpType;
       @Expose @SerializedName("TableProperties")
       private Map<String, String> params;
    +  @Expose @SerializedName("TablePropertyKeys")
    +  private List<String> propertyKeys;
    --- End diff --
    
    It would be better to change this variable to be more intuitive as well. I 
think getter and setter also need to be changed together.


> Add 'ALTER TABLE UNSET PROPERTY' statement to Tajo DDL
> ------------------------------------------------------
>
>                 Key: TAJO-2165
>                 URL: https://issues.apache.org/jira/browse/TAJO-2165
>             Project: Tajo
>          Issue Type: New Feature
>            Reporter: Lee Dongjin
>            Assignee: Lee Dongjin
>
> For version 0.11.x now, there is only set statement for table property. When 
> the user makes a typo with DDL statement, this mistake cannot be removed. We 
> also need the way to remove already set properties.



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

Reply via email to