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

    https://github.com/apache/carbondata/pull/1824#discussion_r162137213
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonPreAggregateRules.scala
 ---
    @@ -347,9 +347,9 @@ case class CarbonPreAggregateQueryRules(sparkSession: 
SparkSession) extends Rule
             }
           // case of handling aggregation query with filter
           case agg@Aggregate(
    -        grExp,
    -        aggExp,
    -        Filter(expression, child@CarbonSubqueryAlias(_, l: 
LogicalRelation)))
    +      grExp,
    --- End diff --
    
    Please add 2 space for all 


---

Reply via email to