Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/1057#discussion_r153935071
--- 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 --
Maybe add some "interesting" datatypes ? Testing integers only may miss some issue.
---
|