[GitHub] carbondata pull request #1469: [WIP] Spark-2.2 Carbon Integration - Phase 1

[ravipesala]

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

    https://github.com/apache/carbondata/pull/1469#discussion_r152571409
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonEnv.scala ---
    @@ -58,7 +59,7 @@ class CarbonEnv {
           ThreadLocalSessionInfo.setCarbonSessionInfo(carbonSessionInfo)
           val config = new CarbonSQLConf(sparkSession)
           
if(sparkSession.conf.getOption(CarbonCommonConstants.ENABLE_UNSAFE_SORT) == 
None) {
    -        config.addDefaultCarbonParams()
    +//        config.addDefaultCarbonParams()
    --- End diff --
    
    remove if not required


---