Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r152582582
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonSession.scala ---
@@ -42,15 +42,46 @@ class CarbonSession(@transient val sc: SparkContext,
this(sc, None)
}
+
+ // SessionStateCodeGenerateFactory.init(sc.version)
+ // CarbonOptimizerCodeGenerateFactory.init(sc.version)
+ // val carbonDefaultOptimizer =
CarbonOptimizerCodeGenerateFactory.getInstance()
+ // .carbonoptimizerFactory.createCarbonOptimizer()
+ // @transient
+ // override lazy val sessionState: SessionState = new
CarbonSessionState(this)
+
+
+
+ def getSessionState(sparkContext: SparkContext): SessionState = {
--- End diff --
move this reflection code to one common util
---