You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by ajantha-bhat <gi...@git.apache.org> on 2018/04/30 02:31:06 UTC

[GitHub] carbondata pull request #2209: [CARBONDATA-2388][SDK]Avro Record Complex Typ...

Github user ajantha-bhat commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2209#discussion_r184905554
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/SchemaReader.java ---
    @@ -81,10 +81,13 @@ public static TableInfo getTableInfo(AbsoluteTableIdentifier identifier)
       }
     
       public static TableInfo inferSchema(AbsoluteTableIdentifier identifier,
    -      boolean isCarbonFileProvider) throws IOException {
    +      boolean isCarbonFileProvider, TableInfo tableInfoFromCache) throws IOException {
         // This routine is going to infer schema from the carbondata file footer
         // Convert the ColumnSchema -> TableSchema -> TableInfo.
         // Return the TableInfo.
    +    if (tableInfoFromCache != null) {
    --- End diff --
    
    If tableInfoFromCache is found, No need to call inferschema itself, please handle this outside infer schema


---