drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-3920) Add vector loading tests
Date Mon, 12 Oct 2015 22:32:05 GMT

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

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

Github user hnfgns commented on a diff in the pull request:

    https://github.com/apache/drill/pull/194#discussion_r41808424
  
    --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/MapVector.java
---
    @@ -355,17 +353,19 @@ public void generateTestData(int values) { }
     
       @Override
       public void clear() {
    -    valueCount = 0;
    -    for (ValueVector v : getChildren()) {
    +    for (final ValueVector v : getChildren()) {
    --- End diff --
    
    the logic in clear() seems duplicating close(). we should directly call close perhaps.


> Add vector loading tests
> ------------------------
>
>                 Key: DRILL-3920
>                 URL: https://issues.apache.org/jira/browse/DRILL-3920
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Execution - Data Types
>    Affects Versions: 1.2.0
>            Reporter: Chris Westin
>            Assignee: Chris Westin
>
> Add some additional tests to TestValueVector to test serialization and reloading of vectors,
as well as the underlying buffer slicing operations that are used for this.



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

Mime
View raw message