Skip to content
Snippets Groups Projects
Commit 38bdc992 authored by Daoyuan Wang's avatar Daoyuan Wang Committed by Reynold Xin
Browse files

[SQL] some comments fix for GROUPING SETS

Author: Daoyuan Wang <daoyuan.wang@intel.com>

Closes #4000 from adrian-wang/comment and squashes the following commits:

9c24fc4 [Daoyuan Wang] some comments
parent 81f72a0d
No related branches found
No related tags found
No related merge requests found
...@@ -126,10 +126,10 @@ class Analyzer(catalog: Catalog, ...@@ -126,10 +126,10 @@ class Analyzer(catalog: Catalog,
} }
/* /*
* GROUP BY a, b, c, WITH ROLLUP * GROUP BY a, b, c WITH ROLLUP
* is equivalent to * is equivalent to
* GROUP BY a, b, c GROUPING SETS ( (a, b, c), (a, b), (a), ( )). * GROUP BY a, b, c GROUPING SETS ( (a, b, c), (a, b), (a), ( ) ).
* Group Count: N + 1 (N is the number of group expression) * Group Count: N + 1 (N is the number of group expressions)
* *
* We need to get all of its subsets for the rule described above, the subset is * We need to get all of its subsets for the rule described above, the subset is
* represented as the bit masks. * represented as the bit masks.
...@@ -139,12 +139,12 @@ class Analyzer(catalog: Catalog, ...@@ -139,12 +139,12 @@ class Analyzer(catalog: Catalog,
} }
/* /*
* GROUP BY a, b, c, WITH CUBE * GROUP BY a, b, c WITH CUBE
* is equivalent to * is equivalent to
* GROUP BY a, b, c GROUPING SETS ( (a, b, c), (a, b), (b, c), (a, c), (a), (b), (c), ( ) ). * GROUP BY a, b, c GROUPING SETS ( (a, b, c), (a, b), (b, c), (a, c), (a), (b), (c), ( ) ).
* Group Count: 2^N (N is the number of group expression) * Group Count: 2 ^ N (N is the number of group expressions)
* *
* We need to get all of its sub sets for a given GROUPBY expressions, the subset is * We need to get all of its subsets for a given GROUPBY expression, the subsets are
* represented as the bit masks. * represented as the bit masks.
*/ */
def bitmasks(c: Cube): Seq[Int] = { def bitmasks(c: Cube): Seq[Int] = {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment