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

    https://github.com/apache/carbondata/pull/2326#discussion_r189788038
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/createTable/TestNonTransactionalCarbonTable.scala
 ---
    @@ -379,6 +377,32 @@ class TestNonTransactionalCarbonTable extends 
QueryTest with BeforeAndAfterAll {
     
         checkExistence(sql("describe formatted sdkOutputTable"), true, "name")
     
    +    buildTestDataWithSortColumns(List())
    +    assert(new File(writerPath).exists())
    +    sql("DROP TABLE IF EXISTS sdkOutputTable")
    +
    +    // with partition
    +    sql(
    +      s"""CREATE EXTERNAL TABLE sdkOutputTable(name string) PARTITIONED BY 
(age int) STORED BY
    +         |'carbondata' LOCATION
    +         |'$writerPath' """.stripMargin)
    +
    +    sql("describe formatted sdkOutputTable").show(false)
    +    sql("select * from sdkOutputTable").show()
    +
    +    buildTestDataWithSortColumns(List(""))
    --- End diff --
    
    in this test case the failure exception should come..so modify the test 
case to intercept for exception here


---

Reply via email to