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

    https://github.com/apache/carbondata/pull/1861#discussion_r165271557
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/timeseries/TestTimeSeriesCreateTable.scala
 ---
    @@ -319,6 +326,53 @@ class TestTimeSeriesCreateTable extends QueryTest with 
BeforeAndAfterAll {
         assert(e.getMessage.equals(s"$timeSeries should define time 
granularity"))
       }
     
    +  test("test timeseries create table 19: should support if not exists") {
    +    sql("DROP DATAMAP IF EXISTS agg1 ON TABLE mainTable")
    +    try {
    --- End diff --
    
    no need for try block. If any exception if thrown the test case will fail


---

Reply via email to