[GitHub] carbondata pull request #2453: [CARBONDATA-2528][MV] Fixed order by in mv an...

[jackylk]

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

    https://github.com/apache/carbondata/pull/2453#discussion_r202224158
  
    --- 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 --
    
    Why is this needed? package visibility problem?


---