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

    https://github.com/apache/carbondata/pull/2960#discussion_r237099347
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/CarbonLateDecodeStrategy.scala
 ---
    @@ -388,9 +388,15 @@ private[sql] class CarbonLateDecodeStrategy extends 
SparkStrategy {
                 }
               case other => other
           }
    +      val updatedColumns = getRequestedColumns(relation,
    --- End diff --
    
    updatedProjects are not used in getRequestedColumn, why use it as a 
parameter and return value


---

Reply via email to