akashrn5 commented on code in PR #4264:
URL: https://github.com/apache/carbondata/pull/4264#discussion_r873345558
##########
integration/spark/src/test/scala/org/apache/carbondata/view/MVTest.scala:
##########
@@ -59,54 +59,54 @@ class MVTest extends QueryTest with BeforeAndAfterAll {
sql("create table source as select * from fact_table")
sql("create materialized view mv1 as select empname, deptname, avg(salary)
from source group by empname, deptname")
val df = sql("select empname, avg(salary) from source group by empname")
- assert(isTableAppearedInPlan(df.queryExecution.optimizedPlan, "mv1"))
+ assert(!isTableAppearedInPlan(df.queryExecution.optimizedPlan, "mv1"))
Review Comment:
this was to test whether hitting MV or not right? in that case need to
enable here and check. Actually since in this test class, in before all better
to make mv enable, so that we can avoid other changes
--
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.
To unsubscribe, e-mail: dev-unsubscr...@carbondata.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
