[GitHub] [carbondata] ajantha-bhat commented on a change in pull request #3753: [CARBONDATA-3810] Partition column name should be case insensitive

[GitBox <gi...@apache.org>]

ajantha-bhat commented on a change in pull request #3753:
URL: https://github.com/apache/carbondata/pull/3753#discussion_r422475259



##########
File path: integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/standardpartition/StandardPartitionTableLoadingTestCase.scala
##########
@@ -519,6 +519,40 @@ class StandardPartitionTableLoadingTestCase extends QueryTest with BeforeAndAfte
     sql("drop table origin_csv")
   }
 
+  test("test partition column case insensitive: insert into") {
+    sql(
+      """create table cs_insert_p
+        |(id int, Name string)
+        |stored as carbondata
+        |partitioned by (c1 int, c2 int, C3 string)""".stripMargin)
+    sql("alter table cs_insert_p drop if exists partition(C1=1, C2=111, c3='2019-11-18')")

Review comment:
       to check case sensitivity, better to have C3 column as some alphabet data instead of date ?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org