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

    https://github.com/apache/spark/pull/21813#discussion_r203917039
  
    --- Diff: sql/core/src/test/resources/sql-tests/inputs/grouping_set.sql ---
    @@ -13,5 +13,39 @@ SELECT a, b, c, count(d) FROM grouping GROUP BY a, b, c 
GROUPING SETS ((a));
     -- SPARK-17849: grouping set throws NPE #3
     SELECT a, b, c, count(d) FROM grouping GROUP BY a, b, c GROUPING SETS 
((c));
     
    +-- Group sets without explicit group by
    +SELECT c1, sum(c2) FROM (VALUES ('x', 10, 0), ('y', 20, 0)) AS t (c1, c2, 
c3) GROUP BY GROUPING SETS (c1);
     
    +-- Group sets without group by and with grouping
    +SELECT c1, sum(c2), grouping(c1) FROM (VALUES ('x', 10, 0), ('y', 20, 0)) 
AS t (c1, c2, c3) GROUP BY GROUPING SETS (c1);
    +
    +-- Mutiple grouping within a grouping set
    +SELECT c1, c2, Sum(c3), grouping__id
    +FROM   (VALUES ('x', 'a', 10), ('y', 'b', 20) ) AS t (c1, c2, c3)
    +GROUP  BY GROUPING SETS ( ( c1 ), ( c2 ) )
    +HAVING GROUPING__ID > 1;
    +
    +-- Group sets without explicit group by
    +SELECT grouping(c1) FROM (VALUES ('x', 'a', 10), ('y', 'b', 20)) AS t (c1, 
c2, c3) GROUP BY c1,c2 GROUPING SETS (c1,c2);
    +
    +-- Mutiple grouping within a grouping set
    +SELECT -c1 AS c1 FROM (values (1,2), (3,2)) t(c1, c2) GROUP BY GROUPING 
SETS ((c1), (c1, c2));
    +
    +-- complex expression in grouping sets
    +SELECT a + b, b, sum(c) FROM (VALUES (1,1,1),(2,2,2)) AS t(a,b,c) GROUP BY 
GROUPING SETS ( (a + b), (b));
    +
    +-- complex expression in grouping sets
    +SELECT a + b, b, sum(c) FROM (VALUES (1,1,1),(2,2,2)) AS t(a,b,c) GROUP BY 
GROUPING SETS ( (a + b), (b + a), (b));
    +
    +-- more query constructs with grouping sets
    +SELECT c1 AS col1, c2 AS col2
    +FROM   (VALUES (1, 2), (3, 2)) t(c1, c2)
    +GROUP  BY GROUPING SETS ( ( c1 ), ( c1, c2 ) )
    +HAVING col2 IS NOT NULL
    +ORDER  BY -col1;
    --- End diff --
    
    @viirya Sorry Simon.. do i have to do something for this comment ?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to