[GitHub] carbondata pull request #2083: [CARBONDATA-2269]Support Query On PreAggregat...

[jackylk]

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

    https://github.com/apache/carbondata/pull/2083#discussion_r177145297
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonPreAggregateRules.scala
 ---
    @@ -296,14 +370,33 @@ case class CarbonPreAggregateQueryRules(sparkSession: 
SparkSession) extends Rule
                       carbonTable,
                       agg)
                   isPlanUpdated = true
    -              Aggregate(updatedGroupExp,
    +              val updateAggPlan = Aggregate(updatedGroupExp,
                     updatedAggExp,
                     CarbonReflectionUtils
                       .getSubqueryAlias(sparkSession,
                         Some(alias1),
                         CarbonReflectionUtils
                           .getSubqueryAlias(sparkSession, Some(alias2), 
newChild, None),
                         None))
    +              if(carbonTable.isStreamingTable) {
    --- End diff --
    
    space after `if`


---