Github user Ben-Zvi commented on the issue:

    https://github.com/apache/drill/pull/1150
  
    @paul-rogers this change also affects the new code (PR#1125) from @ppadma ; 
the required changes are proposed below; are these correct ?   Can you rebase ( 
#1125 should be committed soon), fix, squash, etc. ? 
    
    ```
    --- 
a/exec/java-exec/src/test/java/org/apache/drill/exec/record/TestRecordBatchSizer.java
    +++ 
b/exec/java-exec/src/test/java/org/apache/drill/exec/record/TestRecordBatchSizer.java
    @@ -32,7 +32,7 @@ import org.apache.drill.test.SubOperatorTest;
     import org.apache.drill.test.rowSet.RowSet;
     import org.apache.drill.test.rowSet.RowSet.SingleRowSet;
     import org.apache.drill.test.rowSet.RowSetBuilder;
    -import org.apache.drill.test.rowSet.SchemaBuilder;
    +import org.apache.drill.test.rowSet.schema.SchemaBuilder;
     import org.junit.Test;
    
     public class TestRecordBatchSizer extends SubOperatorTest {
    @@ -506,7 +506,7 @@ public class TestRecordBatchSizer extends 
SubOperatorTest {
           .addMap("map")
             .add("key", MinorType.INT)
             .add("value", MinorType.VARCHAR)
    -      .buildMap()
    +      .resumeSchema()
           .build();
    
         RowSetBuilder builder = fixture.rowSetBuilder(schema);
    @@ -589,7 +589,7 @@ public class TestRecordBatchSizer extends 
SubOperatorTest {
         BatchSchema schema = new SchemaBuilder().addMapArray("map").
           add("key", MinorType.INT).
           add("value", MinorType.VARCHAR).
    -      buildMap().build();
    +      resumeSchema().build();
    
         RowSetBuilder builder = fixture.rowSetBuilder(schema);
    
    @@ -694,8 +694,8 @@ public class TestRecordBatchSizer extends 
SubOperatorTest {
             .addMap("childMap")
               .add("childKey", MinorType.INT)
               .add("childValue", MinorType.VARCHAR)
    -          .buildMap()
    -       .buildMap()
    +          .resumeMap()
    +       .resumeSchema()
           .build();
    
         RowSetBuilder builder = fixture.rowSetBuilder(schema);
    ```


---

Reply via email to