[GitHub] carbondata pull request #1562: [CARBONDATA-1592] Added event listeners

[manishgupta88]

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

    https://github.com/apache/carbondata/pull/1562#discussion_r153192273
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/management/LoadTableCommand.scala
 ---
    @@ -139,6 +140,15 @@ case class LoadTableCommand(
             carbonLoadModel,
             hadoopConf
           )
    +      val operationContext = new OperationContext
    +      val loadTablePreExecutionEvent: LoadTablePreExecutionEvent =
    +        new LoadTablePreExecutionEvent(sparkSession,
    +          null,
    +          carbonLoadModel,
    +          factPath,
    +          dataFrame.isDefined,
    +          optionsFinal)
    --- End diff --
    
    carbonloadmodel is already getting passed... it there any other property 
that is getting used from optionsFinal variable?


---