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

    https://github.com/apache/carbondata/pull/2478#discussion_r204435669
  
    --- Diff: 
datamap/mv/core/src/main/scala/org/apache/carbondata/mv/datamap/MVHelper.scala 
---
    @@ -118,6 +122,43 @@ object MVHelper {
         DataMapStoreManager.getInstance().saveDataMapSchema(dataMapSchema)
       }
     
    +  private def validateMVQuery(sparkSession: SparkSession,
    +      logicalPlan: LogicalPlan) {
    +    val dataMapProvider = DataMapManager.get().getDataMapProvider(null,
    +      new DataMapSchema("", DataMapClassProvider.MV.getShortName), 
sparkSession)
    +    dataMapProvider
    +    var catalog = 
DataMapStoreManager.getInstance().getDataMapCatalog(dataMapProvider,
    +      
DataMapClassProvider.MV.getShortName).asInstanceOf[SummaryDatasetCatalog]
    --- End diff --
    
    Does casting `null` to `SummaryDatasetCatalog` ok?


---

Reply via email to