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

    https://github.com/apache/carbondata/pull/2623#discussion_r211928521
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/carbondata/spark/load/DataLoadProcessBuilderOnSpark.scala
 ---
    @@ -64,6 +65,7 @@ object DataLoadProcessBuilderOnSpark {
         val sortStepRowCounter = sc.accumulator(0, "Sort Processor 
Accumulator")
         val writeStepRowCounter = sc.accumulator(0, "Write Processor 
Accumulator")
     
    +    val confBroadcast = new CarbonConfiguration(hadoopConf)
    --- End diff --
    
    rename variable to confForSerialization. Broadcast name is misleading


---

Reply via email to