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

    https://github.com/apache/carbondata/pull/1481#discussion_r150483161
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/metadata/converter/ThriftWrapperSchemaConverterImpl.java
 ---
    @@ -322,17 +322,22 @@
         for (DataMapSchema wrapperChildSchema : wrapperChildSchemaList) {
           org.apache.carbondata.format.DataMapSchema thriftChildSchema =
               new org.apache.carbondata.format.DataMapSchema();
    -      org.apache.carbondata.format.RelationIdentifier relationIdentifier =
    -          new org.apache.carbondata.format.RelationIdentifier();
    -      relationIdentifier
    -          
.setDatabaseName(wrapperChildSchema.getRelationIdentifier().getDatabaseName());
    -      
relationIdentifier.setTableName(wrapperChildSchema.getRelationIdentifier().getTableName());
    -      
relationIdentifier.setTableId(wrapperChildSchema.getRelationIdentifier().getTableId());
    -      thriftChildSchema.setRelationIdentifire(relationIdentifier);
    +      if (wrapperChildSchema.getRelationIdentifier() != null) {
    +        org.apache.carbondata.format.RelationIdentifier relationIdentifier 
=
    +            new org.apache.carbondata.format.RelationIdentifier();
    +        relationIdentifier
    +            
.setDatabaseName(wrapperChildSchema.getRelationIdentifier().getDatabaseName());
    --- End diff --
    
    ok


---

Reply via email to