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

    https://github.com/apache/carbondata/pull/1672#discussion_r157702204
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/CarbonLateDecodeStrategy.scala
 ---
    @@ -146,27 +184,13 @@ private[sql] class CarbonLateDecodeStrategy extends 
SparkStrategy {
         }
       }
     
    -  protected def pruneFilterProject(
    -      relation: LogicalRelation,
    -      projects: Seq[NamedExpression],
    -      filterPredicates: Seq[Expression],
    -      scanBuilder: (Seq[Attribute], Array[Filter],
    -        ArrayBuffer[AttributeReference]) => RDD[InternalRow]) = {
    -    pruneFilterProjectRaw(
    -      relation,
    -      projects,
    -      filterPredicates,
    -      (requestedColumns, _, pushedFilters, a) => {
    -        scanBuilder(requestedColumns, pushedFilters.toArray, a)
    -      })
    -  }
    -
       protected def pruneFilterProjectRaw(
           relation: LogicalRelation,
           rawProjects: Seq[NamedExpression],
           filterPredicates: Seq[Expression],
    +      partitions: Seq[String],
           scanBuilder: (Seq[Attribute], Seq[Expression], Seq[Filter],
    -        ArrayBuffer[AttributeReference]) => RDD[InternalRow]) = {
    +        ArrayBuffer[AttributeReference], Seq[String]) => RDD[InternalRow]) 
= {
    --- End diff --
    
    identation is not correct


---

Reply via email to