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

    https://github.com/apache/carbondata/pull/2374#discussion_r195680457
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/TableInfo.java
 ---
    @@ -89,6 +90,15 @@
        *
        */
       private boolean isTransactionalTable = true;
    +  /**
    +   * The format of the fact table.
    +   * By default it is carbondata, and we also support other format like CSV
    +   */
    +  private String format = "carbondata";
    --- End diff --
    
    This class should be backward compatible, please make sure you will process 
it if it is null
    And please add @Since("1.4.1") annotation


---

Reply via email to