drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ilooner <...@git.apache.org>
Subject [GitHub] drill pull request #1057: DRILL-5993 Append Row Method For VectorContainer (...
Date Thu, 30 Nov 2017 00:45:13 GMT
Github user ilooner commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1057#discussion_r153958856
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/record/TestVectorContainer.java
---
    @@ -124,4 +132,52 @@ public void testContainerMerge() {
         leftIndirect.clear();
         right.clear();
       }
    +
    +  @Test
    +  public void testAppendRow()
    +  {
    +    MaterializedField colA = MaterializedField.create("colA", Types.required(TypeProtos.MinorType.INT));
    +    MaterializedField colB = MaterializedField.create("colB", Types.required(TypeProtos.MinorType.INT));
    --- End diff --
    
    Added more interesting types. Currently RowSet classes don't support the Map data type.
Paul asked me to look into adding support for this a while ago for DRILL-5870 . I'll update
the test framework to support that in the next PR.


---

Mime
View raw message