[GitHub] carbondata pull request #2478: [CARBONDATA-2540][CARBONDATA-2560][CARBONDATA...

[jackylk]

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

    https://github.com/apache/carbondata/pull/2478#discussion_r204434793
  
    --- 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) {
    --- End diff --
    
    add `: Unit`


---