[
https://issues.apache.org/jira/browse/DRILL-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14304325#comment-14304325
]
Parth Chandra commented on DRILL-2142:
--------------------------------------
+1 non-binding. Don't see anything obviously wrong.
> Refactor complex value vector classes
> -------------------------------------
>
> Key: DRILL-2142
> URL: https://issues.apache.org/jira/browse/DRILL-2142
> Project: Apache Drill
> Issue Type: Bug
> Components: Execution - Data Types
> Reporter: Mehant Baid
> Assignee: Mehant Baid
> Attachments: DRILL-2142.patch
>
>
> Currently we have one base class for the complex vectors.
> AbstractContainerVector is the base class for RepeatedListVector,
> RepeatedMapVector and MapVector. However lot of the functionality related to
> maps in AbstractContainerVector is not needed by RepeatedListVector. In this
> patch I would like to move all the map related logic to a new base class
> called AbstractMapVector which would be used by RepeatedMap and Map classes.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
