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