[
https://issues.apache.org/jira/browse/DRILL-2142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14302066#comment-14302066
]
Mehant Baid commented on DRILL-2142:
------------------------------------
The refactoring you are mentioning is a bigger change and I think warrants a
separate discussion. However this JIRA aims to addresses a slightly different
issue of having logic in AbstractContainerVector that only belongs to Map
related vectors shared by RepeatedListVector. This is holding up one of the
patches related to repeated maps/ maps causing wrong results.
> Refactor complex value vector classes
> -------------------------------------
>
> Key: DRILL-2142
> URL: https://issues.apache.org/jira/browse/DRILL-2142
> Project: Apache Drill
> Issue Type: Bug
> Reporter: Mehant Baid
> Assignee: Mehant Baid
>
> 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)
