VenuReddy2103 commented on a change in pull request #3725:
URL: https://github.com/apache/carbondata/pull/3725#discussion_r415749044



##########
File path: 
index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/secondaryindex/TestSIWithSecondryIndex.scala
##########
@@ -61,6 +62,16 @@ class TestSIWithSecondryIndex extends QueryTest with 
BeforeAndAfterAll {
     }
   }
 
+  test ("test alter drop columns on the SI columns") {
+    sql("create table test1 (name string, id string, country string) stored as 
carbondata")
+    sql("insert into test1 select 'xx', '1', 'china'")
+    sql("create index t_index on table test1(id, country) as 'carbondata'")
+    // alter table to drop the columns used in index
+    sql("alter table test1 drop columns(id, country)")
+    sql("insert into test1 select 'xy'")
+    assert(sql("show indexes on test1").collect().isEmpty)
+  }
+
   test("Test secondry index data count") {

Review comment:
       added




----------------------------------------------------------------
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:
[email protected]


Reply via email to