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

    https://github.com/apache/carbondata/pull/1544#discussion_r154665732
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonPreAggregateRules.scala
 ---
    @@ -328,9 +462,145 @@ case class CarbonPreAggregateQueryRules(sparkSession: 
SparkSession) extends Rule
             Aggregate(updatedGroupExp,
               updatedAggExp,
               newChild)
    +        // case for aggregation query with order by
    +      case Project(_, Sort(sortOrders, global, Aggregate(groupingExp,
    +      aggregateExp,
    +      subQuery@CarbonSubqueryAlias(_, l: LogicalRelation))))
    --- End diff --
    
    Please format it properly



---

Reply via email to