[GitHub] carbondata pull request #2178: show table in presto doesn't initialized load...

[chenliang613]

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

    https://github.com/apache/carbondata/pull/2178#discussion_r200978774
  
    --- Diff: 
integration/presto/src/main/java/org/apache/carbondata/presto/impl/CarbonTableReader.java
 ---
    @@ -314,7 +315,7 @@ private CarbonTable loadTableMetadata(SchemaTableName 
schemaTableName) {
         for (SchemaTableName table : tableList) {
           if (!table.equals(schemaTableName)) continue;
     
    -      return parseCarbonMetadata(table);
    +      return parseCarbonMetadata(table).carbonTable;
    --- End diff --
    
    why here need to add ".carbonTable"? 
    
    Inside parseCarbonMetadata(table), already have the below code 👍 
    result = cache.carbonTable;


---