You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/07/09 04:48:00 UTC

[jira] [Commented] (DRILL-6373) Refactor the Result Set Loader to prepare for Union, List support

    [ https://issues.apache.org/jira/browse/DRILL-6373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16536534#comment-16536534 ] 

ASF GitHub Bot commented on DRILL-6373:
---------------------------------------

paul-rogers commented on issue #1244: DRILL-6373: Refactor Result Set Loader for Union, List support
URL: https://github.com/apache/drill/pull/1244#issuecomment-403358757
 
 
   Rebased on latest paster. Cherry-picked the fix from DRILL-6585 to this branch.
   
   @vrozov or @ppadma - can you kick off a pre-commit build to see if the DRILL-6586 fix resolves the failure we saw earlier? Thanks! 

----------------------------------------------------------------
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:
users@infra.apache.org


> Refactor the Result Set Loader to prepare for Union, List support
> -----------------------------------------------------------------
>
>                 Key: DRILL-6373
>                 URL: https://issues.apache.org/jira/browse/DRILL-6373
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.13.0
>            Reporter: Paul Rogers
>            Assignee: Paul Rogers
>            Priority: Major
>
> As the next step in merging the "batch sizing" enhancements, refactor the {{ResultSetLoader}} and related classes to prepare for Union and List support. This fix follows the refactoring of the column accessors for the same purpose. Actual Union and List support is to follow in a separate PR.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)