Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1071#discussion_r158164670
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/TestUnionDistinct.java ---
    @@ -754,4 +756,37 @@ public void testDrill4147_1() throws Exception {
         }
       }
     
    +  @Test
    +  public void testUnionWithManyColumns() throws Exception {
    --- End diff --
    
    Why would a UNION operator need code for a generated hash table? The union 
matches columns, then iterates over multiple result sets. Where would a 
run-time hash table fit?
    
    Do we need a different unit test that exercises the hash table code?


---

Reply via email to