[GitHub] carbondata pull request #1669: [CARBONDATA-1880] Combine input small files f...

[jackylk]

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

    https://github.com/apache/carbondata/pull/1669#discussion_r157356347
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/carbondata/spark/load/DataLoadProcessBuilderOnSpark.scala
 ---
    @@ -49,29 +59,21 @@ object DataLoadProcessBuilderOnSpark {
       private val LOGGER = 
LogServiceFactory.getLogService(this.getClass.getCanonicalName)
     
       def loadDataUsingGlobalSort(
    -      sc: SparkContext,
    +      sqlContext: SQLContext,
    --- End diff --
    
    better to use sparkSession instead of sqlContext


---