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

    https://github.com/apache/carbondata/pull/2453#discussion_r202244579
  
    --- Diff: 
datamap/mv/core/src/main/scala/org/apache/carbondata/mv/rewrite/SummaryDatasetCatalog.scala
 ---
    @@ -39,6 +40,12 @@ private[mv] case class SummaryDataset(signature: 
Option[Signature],
         dataMapSchema: DataMapSchema,
         relation: ModularPlan)
     
    +case class MVPlanWrapper(plan: ModularPlan, dataMapSchema: DataMapSchema) 
extends ModularPlan {
    --- End diff --
    
    No, it is to provide datamap schema information along with relation. Added 
comment as well.


---

Reply via email to