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

    https://github.com/apache/carbondata/pull/2779#discussion_r221420261
  
    --- Diff: 
integration/spark2/src/main/spark2.3/org/apache/spark/sql/execution/strategy/CarbonDataSourceScan.scala
 ---
    @@ -0,0 +1,55 @@
    +/*
    --- End diff --
    
    I think it is not possible to add lazy variable without breaking spark 2.2 
integration. Since this class is small, I think it is ok to have a separate one 
for spark 2.3 integration.


---

Reply via email to