You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by anubhav100 <gi...@git.apache.org> on 2018/03/06 10:30:23 UTC

[GitHub] carbondata pull request #1990: [CARBONDATA-2195] Add new test case for parti...

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

    https://github.com/apache/carbondata/pull/1990#discussion_r172472894
  
    --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/standardpartition/StandardPartitionTableQueryTestCase.scala ---
    @@ -256,9 +256,11 @@ test("Creation of partition table should fail if the colname in table schema and
           """.stripMargin)
         sql("insert into partitionTable select 1,'huawei','abc'")
         checkAnswer(sql("show partitions partitionTable"), Seq(Row("email=abc")))
    -    intercept[Exception]{
    +    val e = intercept[AnalysisException]{
    --- End diff --
    
    @xubo245 
    sql("alter table partitionTable PARTITION (email='abc') rename to PARTITION (email='def)")
    
    closing single quote is missing after sql("alter table partitionTable PARTITION (email='def) rename to PARTITION (email='def)") it is going to pass anyhow
    



---