Github user amansinha100 commented on the issue:

    https://github.com/apache/drill/pull/1106
  
    Seems ok to fix the RecordBatchLoader.isSameSchema() since it is missing 
the checks for nested columns.  In order to do the consolidation that Paul 
suggested,  you might want to open an enhancement JIRA.  Since 
MaterializedField and RecordBatchLoader have separate class hierarchies, 
there's not a direct way to have a single method do the comparison.   One 
option is to create a utility 'SchemaComparator' class that incorporates 
various static utility methods and Javadoc the rules for the comparisons. 
    
    So, I am +1  on this change. 


---

Reply via email to