[GitHub] carbondata pull request #2484: [WIP] added hadoop conf to thread local

[gvramana]

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

    https://github.com/apache/carbondata/pull/2484#discussion_r201614265
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/carbondata/spark/rdd/CarbonRDD.scala
 ---
    @@ -46,18 +46,12 @@ abstract class CarbonRDD[T: ClassTag](@transient sc: 
SparkContext,
           info = new CarbonSessionInfo
           info.setSessionParams(new SessionParams())
         }
    +    info.getThreadParams
    --- End diff --
    
    1. Here configuration should be taken from SparkSession,sessionState
    2. Remove hadoopConf from constructor


---