carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jackylk <...@git.apache.org>
Subject [GitHub] carbondata pull request #1484: [CARBONDATA-1700][DataLoad] Add TableProperti...
Date Sun, 12 Nov 2017 15:03:22 GMT
Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1484#discussion_r150413184
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/table/TableSchema.java
---
    @@ -77,6 +78,7 @@
     
       public TableSchema() {
         this.listOfColumns = new ArrayList<ColumnSchema>(CarbonCommonConstants.DEFAULT_COLLECTION_SIZE);
    +    this.tableProperties = new HashMap<String, String>(5);
    --- End diff --
    
    I think it is ok to use default constructor of HashMap, 5 is not needed


---

Mime
View raw message