luocooong commented on PR #2515:
URL: https://github.com/apache/drill/pull/2515#issuecomment-1108226622

   > Found the bug. It is in `ColumnBuilder` which seems to be missing code to 
handle an unprojected repeated list. This bug then caused the other "bugs" that 
we discussed in the review: those bits of code are working as they should. The 
problem is that the result set loader is materializing a vector when it should 
not. It will take some time to remember how all this stuff works. Stay tuned.
   
   @paul-rogers Great! Thank you for the quick work.
   From the end of the most recent discussion, I completely rejected my 
previous code revision, guess that the unprojected handle might have been lost. 
Actually, I've added this function locally, but I'm not sure it's correct. 
Would you mind checking mine before you submit the new revision?


-- 
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

Reply via email to