You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2018/07/10 15:42:46 UTC

[GitHub] parthchandra commented on a change in pull request #1358: DRILL-6516: EMIT support in streaming agg

parthchandra commented on a change in pull request #1358:  DRILL-6516: EMIT support in streaming agg
URL: https://github.com/apache/drill/pull/1358#discussion_r201170576
 
 

 ##########
 File path: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/lateraljoin/TestE2EUnnestAndLateral.java
 ##########
 @@ -394,4 +394,47 @@ public void testLateral_HashAgg_with_nulls() throws Exception {
       .baselineValues("dd",222L)
       .build().run();
   }
+
+  @Test
+  public void testMultipleBatchesLateral_WithStreamingAgg() throws Exception {
+    String sql = "SELECT t2.maxprice FROM (SELECT customer.c_orders AS c_orders FROM "
+        + "dfs.`lateraljoin/multipleFiles/` customer) t1, LATERAL (SELECT CAST(MAX(t.ord.o_totalprice)"
+        + " AS int) AS maxprice FROM UNNEST(t1.c_orders) t(ord) GROUP BY t.ord.o_orderstatus) t2";
+
+    testBuilder()
+        .optionSettingQueriesForTestQuery("alter session set `%s` = true",
+            PlannerSettings.STREAMAGG.getOptionName())
 
 Review comment:
   Had to do this because the HashAgg tests were disabling the streaming agg. Removed this and reset the option in the HashAgg tests.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services