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

    https://github.com/apache/drill/pull/1059#discussion_r158356421
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/physical/unit/TestNullInputMiniPlan.java
 ---
    @@ -323,8 +323,6 @@ public void testHashJoinLeftEmpty() throws Exception {
             .build();
     
         BatchSchema expectedSchema = new SchemaBuilder()
    -        .addNullable("a", TypeProtos.MinorType.BIGINT)
    -        .addNullable("b", TypeProtos.MinorType.BIGINT)
    --- End diff --
    
    @paul-rogers Thank you for the review. I have changed the code to make this 
case work. I have made changes to incorporate the earlier comments. Can you 
please review the changes.


---

Reply via email to