[GitHub] carbondata pull request #2818: [CARBONDATA-3011] Add carbon property to conf...

[ravipesala]

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

    https://github.com/apache/carbondata/pull/2818#discussion_r227708688
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/CarbonLateDecodeStrategy.scala
 ---
    @@ -374,10 +410,27 @@ private[sql] class CarbonLateDecodeStrategy extends 
SparkStrategy {
             metadata,
             needDecoder,
             updateRequestedColumns.asInstanceOf[Seq[Attribute]])
    -      execution.ProjectExec(
    -        updateRequestedColumnsFunc(updatedProjects, table,
    -          needDecoder).asInstanceOf[Seq[NamedExpression]],
    -        filterCondition.map(execution.FilterExec(_, scan)).getOrElse(scan))
    +      // Check whether spark should handle row filters in case of vector 
flow.
    +      if (!vectorPushRowFilters && scan.isInstanceOf[CarbonDataSourceScan]
    +          && !implictsExisted && !hasDictionaryFilterCols) {
    --- End diff --
    
    updated in comment


---