diff options
| author | Liang-Chi Hsieh <viirya@gmail.com> | 2015-04-13 18:15:29 -0700 |
|---|---|---|
| committer | Michael Armbrust <michael@databricks.com> | 2015-04-13 18:15:29 -0700 |
| commit | 5b8b324f33e857b95de65031334846a7ca26fa60 (patch) | |
| tree | 5217378a4194ccc9414f07adc2df278e334812d6 /sbin | |
| parent | d7f2c198678d1f5ffc1c6da3a6fb189b0a4ef070 (diff) | |
| download | spark-5b8b324f33e857b95de65031334846a7ca26fa60.tar.gz spark-5b8b324f33e857b95de65031334846a7ca26fa60.tar.bz2 spark-5b8b324f33e857b95de65031334846a7ca26fa60.zip | |
[SPARK-6303][SQL] Remove unnecessary Average in GeneratedAggregate
Because `Average` is a `PartialAggregate`, we never get a `Average` node when reaching `HashAggregation` to prepare `GeneratedAggregate`.
That is why in SQLQuerySuite there is already a test for `avg` with codegen. And it works.
But we can find a case in `GeneratedAggregate` to deal with `Average`. Based on the above, we actually never execute this case.
So we can remove this case from `GeneratedAggregate`.
Author: Liang-Chi Hsieh <viirya@gmail.com>
Closes #4996 from viirya/add_average_codegened and squashes the following commits:
621c12f [Liang-Chi Hsieh] Merge remote-tracking branch 'upstream/master' into add_average_codegened
368cfbc [Liang-Chi Hsieh] Merge remote-tracking branch 'upstream/master' into add_average_codegened
74926d1 [Liang-Chi Hsieh] Add Average in canBeCodeGened lists.
Diffstat (limited to 'sbin')
0 files changed, 0 insertions, 0 deletions