[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-19 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r597429391 ## File path: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonTableCompactor.scala ## @@ -276,7 +294,40 @@ class

[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-18 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r597398064 ## File path: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonTableCompactor.scala ## @@ -276,7 +290,25 @@ class

[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-18 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r597396979 ## File path: index/secondary-index/src/test/scala/org/apache/carbondata/spark/testsuite/secondaryindex/TestSIWithPartition.scala ## @@ -380,6

[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-18 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r597396875 ## File path: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonMergerRDD.scala ## @@ -98,6 +98,24 @@ class

[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-18 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r596995330 ## File path: integration/spark/src/main/scala/org/apache/spark/sql/execution/command/partition/CarbonAlterTableAddHivePartitionCommand.scala ##

[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-18 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r596995121 ## File path: core/src/main/java/org/apache/carbondata/core/readcommitter/TableStatusReadCommittedScope.java ## @@ -86,6 +87,13 @@ public

[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-18 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r596994614 ## File path: core/src/main/java/org/apache/carbondata/core/indexstore/ExtendedBlockletWrapper.java ## @@ -121,8 +121,9 @@ public

[GitHub] [carbondata] ShreelekhyaG commented on a change in pull request #4107: [CARBONDATA-4149] Query with SI after add partition based on location on partition table gives incorrect results

2021-03-18 Thread GitBox
ShreelekhyaG commented on a change in pull request #4107: URL: https://github.com/apache/carbondata/pull/4107#discussion_r596992325 ## File path: core/src/main/java/org/apache/carbondata/core/index/Segment.java ## @@ -417,4 +417,8 @@ public void