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

    https://github.com/apache/carbondata/pull/1672#discussion_r157703591
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/CarbonLateDecodeStrategy.scala
 ---
    @@ -130,6 +130,44 @@ private[sql] class CarbonLateDecodeStrategy extends 
SparkStrategy {
           table.carbonTable.getTableInfo.serialize())
       }
     
    +  /**
    +   * Converts to physical RDD of carbon after pushing down applicable 
filters.
    +   * @param relation
    +   * @param projects
    +   * @param filterPredicates
    +   * @param scanBuilder
    --- End diff --
    
    can you add comment for these parameter


---

Reply via email to