You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by jackylk <gi...@git.apache.org> on 2017/12/06 15:04:23 UTC

[GitHub] carbondata pull request #1614: [CARBONDATA-1592] Added Event Listeners & Rem...

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

    https://github.com/apache/carbondata/pull/1614#discussion_r155261399
  
    --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/events/Events.scala ---
    @@ -93,12 +93,20 @@ trait AlterTableAddColumnEventInfo {
     /**
      * event for alter_table_rename
      */
    -trait AlterTableCompactionEventInfo {
    +trait AlterTableCompactionStatusUpdateEventInfo {
       val carbonTable: CarbonTable
       val carbonLoadModel: CarbonLoadModel
       val mergedLoadName: String
     }
     
    +/**
    + * event for alter_table_compaction
    + */
    +trait AlterTableCompactionEventInfo {
    --- End diff --
    
    AlterTableCompactionPostEvent has sparkSession and carbonTable already, why this trait is required


---