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

    https://github.com/apache/carbondata/pull/2448#discussion_r200258280
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonDropDataMapCommand.scala
 ---
    @@ -201,12 +201,12 @@ case class CarbonDropDataMapCommand(
             val operationContext: OperationContext = new OperationContext()
             val systemFolderLocation: String = 
CarbonProperties.getInstance().getSystemFolderLocation
             val updateDataMapPreExecutionEvent: UpdateDataMapPreExecutionEvent 
=
    -          UpdateDataMapPreExecutionEvent(sparkSession, 
systemFolderLocation)
    +          UpdateDataMapPreExecutionEvent(sparkSession, 
systemFolderLocation, table.get)
             
OperationListenerBus.getInstance().fireEvent(updateDataMapPreExecutionEvent,
               operationContext)
             DataMapStatusManager.dropDataMap(dataMapSchema.getDataMapName)
             val updateDataMapPostExecutionEvent: 
UpdateDataMapPostExecutionEvent =
    -          UpdateDataMapPostExecutionEvent(sparkSession, 
systemFolderLocation)
    +          UpdateDataMapPostExecutionEvent(sparkSession, 
systemFolderLocation,table.get)
    --- End diff --
    
    need a space after comma


---

Reply via email to