ppadma commented on a change in pull request #1429: DRILL-6676: Add Union, List and Repeated List types to Result Set Loader URL: https://github.com/apache/drill/pull/1429#discussion_r211062745
########## File path: exec/vector/src/main/java/org/apache/drill/exec/vector/complex/BaseRepeatedValueVector.java ########## @@ -212,6 +212,18 @@ protected void replaceDataVector(ValueVector v) { vector = v; } + public void setChildVector(ValueVector childVector) { + + // When created, the list uses the default vector of type LATE. + // That entry appears as a child vector. Remove it and add the + // new type instead. + + assert vector == DEFAULT_DATA_VECTOR; + replaceDataVector(childVector); Review comment: do we want to check if childVector is valid i.e. not null and any other checks ? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services