[GitHub] carbondata pull request #2552: [CARBONDATA-2781] Added fix for Null Pointer ...

[ravipesala]

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

    https://github.com/apache/carbondata/pull/2552#discussion_r206458961
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonCreateDataMapCommand.scala
 ---
    @@ -181,7 +181,18 @@ case class CarbonCreateDataMapCommand(
     
       override def undoMetadata(sparkSession: SparkSession, exception: 
Exception): Seq[Row] = {
         if (dataMapProvider != null) {
    -      dataMapProvider.cleanMeta()
    +      if (dataMapSchema.getProviderName
    --- End diff --
    
    I think datamap drop is required for all types of datamap , not just 
preaggregate


---