ajantha-bhat commented on a change in pull request #3787: URL: https://github.com/apache/carbondata/pull/3787#discussion_r486289292
########## File path: index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/secondaryindex/TestSIWithSecondryIndex.scala ########## @@ -86,6 +86,64 @@ class TestSIWithSecondryIndex extends QueryTest with BeforeAndAfterAll { .contains("Alter table drop column operation failed:")) } + test("test create secondary index global sort after insert") { + sql("drop table if exists table1") + sql("create table table1 (name string, id string, country string) stored as carbondata") + sql("insert into table1 select 'xx', '2', 'china' union all select 'xx', '1', 'india'") + sql("create index table1_index on table table1(id, country) as 'carbondata' properties" + + "('sort_scope'='global_sort', 'Global_sort_partitions'='3')") + checkAnswerWithoutSort(sql("select id, country from table1_index"), + Seq(Row("1", "india"), Row("2", "china"))) + // check for valid sort_scope + checkExistence(sql("describe formatted table1_index"), true, "Sort Scope global_sort") + // check the invalid sort scope + assert(intercept[MalformedCarbonCommandException](sql( + "create index index_2 on table table1(id, country) as 'carbondata' properties" + + "('sort_scope'='tim_sort', 'Global_sort_partitions'='3')")) + .getMessage + .contains("Invalid SORT_SCOPE tim_sort")) + // check for invalid global_sort_partitions + assert(intercept[MalformedCarbonCommandException](sql( + "create index index_2 on table table1(id, country) as 'carbondata' properties" + + "('sort_scope'='global_sort', 'Global_sort_partitions'='-1')")) + .getMessage + .contains("Table property global_sort_partitions : -1 is invalid")) + sql("drop index table1_index on table1") Review comment: I can see that from your PR #3608 , you have used t1 as table name and i1 as index name in DropTableTest. is that clean and meaningful name ? I don't want to argue further. Table1 is still a table name I have not named it as car or bike. ---------------------------------------------------------------- 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