[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #3957: [WIP] drop index table is failed with table not found exception

2020-09-25 Thread GitBox


Indhumathi27 commented on a change in pull request #3957:
URL: https://github.com/apache/carbondata/pull/3957#discussion_r494856214



##
File path: 
index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/secondaryindex/DropTableTest.scala
##
@@ -74,4 +74,17 @@ class DropTableTest extends QueryTest with BeforeAndAfterAll 
{
 assert(exception.getMessage.contains("Index with name indtestdrop does not 
exist"))
 sql("drop table if exists testDrop")
   }
+
+  test("test drop index command after refresh the index for empty index 
table") {
+sql("drop table if exists testDropIndex")
+sql("create table testDropIndex (a string, b string, c string) STORED AS 
carbondata")
+sql("create index helloIndex on table testDropIndex (c) AS 'carbondata' 
properties" +
+"('table_blocksize'='1')")
+sql("show indexes on testDropIndex").show()

Review comment:
   please remove this line





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:
us...@infra.apache.org




[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #3957: [WIP] drop index table is failed with table not found exception

2020-09-25 Thread GitBox


Indhumathi27 commented on a change in pull request #3957:
URL: https://github.com/apache/carbondata/pull/3957#discussion_r494856214



##
File path: 
index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/secondaryindex/DropTableTest.scala
##
@@ -74,4 +74,17 @@ class DropTableTest extends QueryTest with BeforeAndAfterAll 
{
 assert(exception.getMessage.contains("Index with name indtestdrop does not 
exist"))
 sql("drop table if exists testDrop")
   }
+
+  test("test drop index command after refresh the index for empty index 
table") {
+sql("drop table if exists testDropIndex")
+sql("create table testDropIndex (a string, b string, c string) STORED AS 
carbondata")
+sql("create index helloIndex on table testDropIndex (c) AS 'carbondata' 
properties" +
+"('table_blocksize'='1')")
+sql("show indexes on testDropIndex").show()

Review comment:
   please remove this line





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:
us...@infra.apache.org