You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2019/11/19 19:50:59 UTC

[GitHub] [drill] paul-rogers commented on issue #1899: DRILL-7445: Create batch copier based on result set framework

paul-rogers commented on issue #1899: DRILL-7445: Create batch copier based on result set framework
URL: https://github.com/apache/drill/pull/1899#issuecomment-555680733
 
 
   @ihuzenko, thanks for your detailed review. Addressed most of your comments, added explanations for the others.
   
   To provide a bit of context: this is a first step toward additional cleanup in the operators ("record batches") based on issues I've been finding as I work though the batch count, row count, vector value count issues. Did this change as its own PR to make it easier to review. A subsequent PR will use these changes to modify selected operators. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services