Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1857#discussion_r178033617
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/preaggregate/TestPreAggCreateCommand.scala
---
@@ -55,12 +57,33 @@ class TestPreAggCreateCommand extends QueryTest with
BeforeAndAfterAll {
}
test("test pre agg create table 2") {
+ dropDataMaps("PreAggMain", "preagg2")
sql("create datamap preagg2 on table PreAggMain using 'preaggregate'
as select a as a1,sum(b) as udfsum from PreAggMain group by a")
checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg2"), true,
"preaggmain_a")
checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg2"), true,
"preaggmain_b_sum")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg2"), false,
"preaggmain_a1")
sql("drop datamap preagg2 on table PreAggMain")
}
+ test("test pre agg create table 3") {
+ dropDataMaps("PreAggMain", "preagg3")
+ sql("create datamap preagg3 on table PreAggMain using 'preaggregate'
as select a,sum(b) as sum from PreAggMain group by a")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg3"), true,
"preaggmain_a")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg3"), true,
"preaggmain_b_sum")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg3"), false,
"preaggmain_sum")
+ sql("drop datamap preagg3 on table PreAggMain")
+ }
+
+ test("test pre agg create table 4") {
+ dropDataMaps("PreAggMain", "preagg4")
+ sql("create datamap preagg4 on table PreAggMain using 'preaggregate'
as select a as a1,sum(b) as sum from PreAggMain group by a")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg4"), true,
"preaggmain_a")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg4"), true,
"preaggmain_b_sum")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg4"), false,
"preaggmain_a1")
+ checkExistence(sql("DESCRIBE FORMATTED PreAggMain_preagg4"), false,
"preaggmain_sum")
--- End diff --
this check can be moved to "test pre agg create table 2"
---
