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
