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