ajantha-bhat commented on a change in pull request #4106:
URL: https://github.com/apache/carbondata/pull/4106#discussion_r596560888
##########
File path:
integration/spark/src/test/scala/org/apache/carbondata/view/rewrite/TestPartitionWithMV.scala
##########
@@ -748,6 +748,22 @@ class TestPartitionWithMV extends QueryTest with
BeforeAndAfterAll with BeforeAn
sql("drop table if exists partitionone")
}
+ test("test partition on MV with sort column") {
+ sql("drop table if exists partitionone")
+ sql("create table if not exists partitionone (ts timestamp, " +
+ "metric STRING, tags_id STRING, value DOUBLE) partitioned by (ts1
timestamp,ts2 timestamp) stored as carbondata TBLPROPERTIES
('SORT_COLUMNS'='metric,ts2')")
Review comment:
I think above MV rearrange code was added when partition column is not a
sort column case I guess.
can you please add one more test case, same but just remove sort columns
from the create table and test it ?
----------------------------------------------------------------
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
