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

    https://github.com/apache/carbondata/pull/1856#discussion_r174064354
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/timeseries/TestTimeseriesDataLoad.scala
 ---
    @@ -167,59 +170,60 @@ class TestTimeseriesDataLoad extends QueryTest with 
BeforeAndAfterAll {
            """.stripMargin)
     
       }
    -  test("test Year level timeseries data validation1 ") {
    -    checkAnswer( sql("select count(*) from table_03_ag1_year"),
    +
    +  test("test timeseries table selection 1: year level timeseries data 
validation1 ") {
    +    checkAnswer(sql("SELECT COUNT(*) FROM table_03_ag1_year"),
           Seq(Row(4)))
       }
     
    -  test("test month level timeseries data validation1 ") {
    -    checkAnswer( sql("select count(*) from table_03_ag1_month"),
    +  test("test timeseries table selection 2: month level timeseries data 
validation1 ") {
    +    checkAnswer(sql("SELECT COUNT(*) FROM table_03_ag1_month"),
           Seq(Row(4)))
       }
     
    -  test("test day level timeseries data validation1 ") {
    -    checkAnswer( sql("select count(*) from table_03_ag1_day"),
    +  test("test timeseries table selection 3: day level timeseries data 
validation1 ") {
    +    checkAnswer(sql("SELECT COUNT(*) FROM table_03_ag1_day"),
    --- End diff --
    
    why is this required?


---

Reply via email to