Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2083#discussion_r177145680
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonPreAggregateRules.scala
---
@@ -466,7 +594,7 @@ case class CarbonPreAggregateQueryRules(sparkSession:
SparkSession) extends Rule
carbonTable,
agg)
isPlanUpdated = true
- Aggregate(updatedGroupExp,
+ val updateAggPlan = Aggregate(updatedGroupExp,
--- End diff --
move `updatedGroupExp` to next line
---