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

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

Github user parthchandra commented on the pull request:

    https://github.com/apache/drill/pull/250#issuecomment-155632914
  
    In general the patch looks fine.  I'm not sure I follow the reason for the 
drill-logical package.
    My one concern was similar to Hakim's that this would affect the allocator 
changes quite a bit (and I think the allocator changes add more value). Can we 
split the changes? @adeneche can you comment on what changes would make it 
easier for you to merge?
    BTW, master has moved and there is a merge conflict (easily resolved). 


> Create a POC VV extraction
> --------------------------
>
>                 Key: DRILL-3987
>                 URL: https://issues.apache.org/jira/browse/DRILL-3987
>             Project: Apache Drill
>          Issue Type: Sub-task
>            Reporter: Jacques Nadeau
>            Assignee: Jacques Nadeau
>             Fix For: 1.4.0
>
>
> I'd like to start by looking at an extraction that pulls out the base 
> concepts of:
> buffer allocation, value vectors and complexwriter/fieldreader.
> I need to figure out how to resolve some of the cross-dependency issues (such 
> as the jdbc accessor connections).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to