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?
---
