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

    https://github.com/apache/carbondata/pull/2423#discussion_r199315133
  
    --- Diff: 
datamap/mv/core/src/main/scala/org/apache/carbondata/mv/datamap/MVAnalyzerRule.scala
 ---
    @@ -75,6 +77,13 @@ class MVAnalyzerRule(sparkSession: SparkSession) extends 
Rule[LogicalPlan] {
             plan
           }
         } else {
    +      if (catalog != null && (plan.isInstanceOf[InsertIntoCarbonTable]
    --- End diff --
    
    please move `(plan.isInstanceOf[InsertIntoCarbonTable]` to next line


---

Reply via email to