Indhumathi27 commented on a change in pull request #3786:
URL: https://github.com/apache/carbondata/pull/3786#discussion_r440035757



##########
File path: 
integration/spark/src/test/scala/org/apache/carbondata/view/rewrite/TestAllOperationsOnMV.scala
##########
@@ -610,10 +610,10 @@ class TestAllOperationsOnMV extends QueryTest with 
BeforeAndAfterEach {
     sql("drop materialized view if exists mv1")
     sql("create materialized view mv1  as select a.name,a.price from maintable 
a")
     var dataFrame = sql("select a.name,a.price from maintable a limit 1")
-    assert(dataFrame.count() == 1)

Review comment:
       Here, count API is not applying collect rows on SPARK PHYSICAL PLAN. 
When i checked it, plan was on main table




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to