raghavyadav01 commented on code in PR #17396:
URL: https://github.com/apache/pinot/pull/17396#discussion_r2633194092
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/loader/invertedindex/TextIndexHandler.java:
##########
@@ -127,6 +127,9 @@ public void updateIndices(SegmentDirectory.Writer
segmentWriter)
Set<String> existingColumns =
segmentWriter.toSegmentDirectory().getColumnsWithIndex(StandardIndexes.text());
// Handle configuration changes for existing indexes
for (String column : existingColumns) {
+ if (!columnsToAddIdx.contains(column)) {
Review Comment:
When column was configured with text index and text index is removed from
column , we do not need to check if index creation is required.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
