carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From manishgupta88 <...@git.apache.org>
Subject [GitHub] carbondata pull request #1562: [CARBONDATA-1592] Added event listeners
Date Mon, 27 Nov 2017 13:15:06 GMT
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?


---

Mime
View raw message