[GitHub] carbondata pull request #1469: [CARBONDATA-1552][Spark-2.2 Integration] Spar...

[jackylk]

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

    https://github.com/apache/carbondata/pull/1469#discussion_r153104066
  
    --- Diff: integration/spark2/src/main/spark2.1/CarbonSessionState.scala ---
    @@ -172,6 +173,10 @@ class CarbonSessionState(sparkSession: SparkSession) 
extends HiveSessionState(sp
           conf,
           newHadoopConf())
       }
    +
    +  def getCarbonEnv() : CarbonEnv = {
    +    catalog.carbonEnv
    --- End diff --
    
    Why make it session based instead of a global CarbonEnv?


---