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

    https://github.com/apache/carbondata/pull/1833#discussion_r162600231
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/standardpartition/StandardPartitionTableOverwriteTestCase.scala
 ---
    @@ -155,9 +155,28 @@ class StandardPartitionTableOverwriteTestCase extends 
QueryTest with BeforeAndAf
         checkAnswer(sql("select count(*) from weather6"), Seq(Row(2)))
       }
     
    +  test("Test overwrite static partition with wrong int value") {
    +    sql(
    +      """
    +        | CREATE TABLE weather7 (type String)
    +        | PARTITIONED BY (year int, month int, day int)
    +        | STORED BY 'org.apache.carbondata.format'
    +      """.stripMargin)
    +
    +    sql("insert into weather7 partition(year=2014, month=05, day=25) 
select 'rainy'")
    +    sql("insert into weather7 partition(year=2014, month=04, day=23) 
select 'cloudy'")
    +    sql("insert overwrite table weather7 partition(year=2014, month=05, 
day=25) select 'sunny'")
    +    checkExistence(sql("select * from weather7"), true, "sunny")
    +    checkAnswer(sql("select count(*) from weather7"), Seq(Row(2)))
    +    sql("insert into weather7 partition(year=2014, month, day) select 
'rainy1',06,25")
    +    sql("insert into weather7 partition(year=2014, month=01, day) select 
'rainy2',27")
    +    sql("insert into weather7 partition(year=2014, month=01, day=02) 
select 'rainy3'")
    +    checkAnswer(sql("select count(*) from weather7 where month=1"), 
Seq(Row(2)))
    +  }
    +
     
       override def afterAll = {
    -    dropTable
    +//    dropTable
    --- End diff --
    
    Please uncomment this line


---

Reply via email to