[GitHub] carbondata pull request #1605: [CARBONDATA-1526] [PreAgg] Added support to c...

[jackylk]

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

    https://github.com/apache/carbondata/pull/1605#discussion_r154964861
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/carbondata/spark/rdd/Compactor.scala
 ---
    @@ -69,11 +69,7 @@ object Compactor {
         // trigger event for compaction
         val operationContext = new OperationContext
         val alterTableCompactionPreEvent: AlterTableCompactionPreEvent =
    -      
AlterTableCompactionPreEvent(compactionCallableModel.sqlContext.sparkSession,
    -        carbonTable,
    -        carbonLoadModel,
    -        mergedLoadName,
    -        sc)
    +      AlterTableCompactionPreEvent(carbonTable, carbonMergerMapping, 
mergedLoadName, sc)
    --- End diff --
    
    please modify line 55, it is not storePath, it is tablePath


---