luocooong commented on code in PR #2515:
URL: https://github.com/apache/drill/pull/2515#discussion_r851627884
##########
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/scan/v3/schema/MutableTupleSchema.java:
##########
@@ -173,7 +173,7 @@ public ColumnHandle insert(int posn, ColumnMetadata col) {
}
public ColumnHandle insert(ColumnMetadata col) {
- return insert(insertPoint++, col);
+ return insert(insertPoint == -1 ? size() : insertPoint++, col);
Review Comment:
When I fixed the following error, I received a new one : Dynamically create
a repeated list field (and success) in next(), then this field may be assigned
to the index value of -1.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
