You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by chenliang613 <gi...@git.apache.org> on 2018/07/09 12:11:06 UTC

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

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;


---