[ https://issues.apache.org/jira/browse/DRILL-3313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14646739#comment-14646739 ]
ASF GitHub Bot commented on DRILL-3313: --------------------------------------- Github user jaltekruse commented on the pull request: https://github.com/apache/drill/pull/81#issuecomment-126091654 Apologies for the forthcoming double comments. I forgot to make comments on the pull request and instead was viewing the individual commit. These comments are nicely aggregated on the github IU, but are not currently picked up the the JIRA bot to copy the comments to the issue report. I will be duplicating my comments on the same lines they were on the individual commit on the pull request summary diff to make sure they get copied over. > Eliminate redundant #load methods and unit-test loading & exporting of vectors > ------------------------------------------------------------------------------ > > Key: DRILL-3313 > URL: https://issues.apache.org/jira/browse/DRILL-3313 > Project: Apache Drill > Issue Type: Sub-task > Components: Execution - Data Types > Affects Versions: 1.0.0 > Reporter: Hanifi Gunes > Assignee: Jason Altekruse > Fix For: 1.2.0 > > > Vectors have multiple #load methods that are used to populate data from raw > buffers. It is relatively tough to reason, maintain and unit-test loading and > exporting of data since there is many redundant code around load methods. > This issue proposes to have single #load method conforming to VV#load(def, > buffer) signature eliminating all other #load overrides. -- This message was sent by Atlassian JIRA (v6.3.4#6332)