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

    https://github.com/apache/carbondata/pull/1605#discussion_r154962941
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/carbondata/events/AlterTableEvents.scala
 ---
    @@ -133,25 +133,25 @@ case class AlterTableRenameAbortEvent(carbonTable: 
CarbonTable,
     /**
      *
      * @param carbonTable
    - * @param carbonLoadModel
    + * @param carbonMergerMapping
      * @param mergedLoadName
      * @param sQLContext
      */
    -case class AlterTableCompactionPreEvent(sparkSession: SparkSession, 
carbonTable: CarbonTable,
    -    carbonLoadModel: CarbonLoadModel,
    +case class AlterTableCompactionPreEvent(carbonTable: CarbonTable,
    +    carbonMergerMapping: CarbonMergerMapping,
         mergedLoadName: String,
         sQLContext: SQLContext) extends Event with 
AlterTableCompactionEventInfo
     
     
     /**
      *
      * @param carbonTable
    - * @param carbonLoadModel
    + * @param carbonMergerMapping
    --- End diff --
    
    Add description of all these parameters, same for all related events


---

Reply via email to