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

    https://github.com/apache/carbondata/pull/1694#discussion_r158931680
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/preaggregate/TestPreAggregateExpressions.scala
 ---
    @@ -0,0 +1,44 @@
    +package org.apache.carbondata.integration.spark.testsuite.preaggregate
    +
    +import org.apache.spark.sql.test.util.QueryTest
    +import org.scalatest.BeforeAndAfterAll
    +
    +class TestPreAggregateExpressions extends QueryTest with BeforeAndAfterAll 
{
    +
    +  override def beforeAll: Unit = {
    +    sql("drop table if exists mainTable")
    +    sql("CREATE TABLE mainTable(id int, name string, city string, age 
string) STORED BY 'org.apache.carbondata.format'")
    +    sql("create datamap agg0 on table mainTable using 'preaggregate' as 
select name,count(age) from mainTable group by name")
    +    sql("create datamap agg1 on table mainTable using 'preaggregate' as 
select name,sum(case when age=35 then id else 0 end) from mainTable group by 
name")
    +    sql("create datamap agg2 on table mainTable using 'preaggregate' as 
select name,sum(case when age=35 then id else 0 end),city from mainTable group 
by name,city")
    +    sql("create datamap agg3 on table mainTable using 'preaggregate' as 
select name,sum(case when age=27 then id else 0 end) from mainTable group by 
name")
    --- End diff --
    
    I think these `CREATE DATAMAP` statement should be run inside the test 
function, not in beforeAll


---

Reply via email to