eldenmoon commented on code in PR #69761:
URL: https://github.com/apache/doris/pull/69761#discussion_r4438732465
##########
fe/fe-core/src/main/java/org/apache/doris/catalog/OlapTable.java:
##########
@@ -4240,6 +4248,19 @@ public Index getInvertedIndex(Column column,
List<String> subPath, String analyz
}
}
+ private List<Index> getInvertedIndexes(Column column) {
+ List<Index> invertedIndexes = new ArrayList<>();
+ for (Index index : indexes.getIndexes()) {
+ if (index.getIndexType() == IndexType.INVERTED) {
Review Comment:
Fixed in c7fec9b1ff7. `OlapTable.getInvertedIndexes()` now compares the
index column with `equalsIgnoreCase`, so a column `V` with `INDEX
values_idx(v)` binds the values index for both `V MATCH_ANY` and `v MATCH_ANY`;
column names are unique case-insensitively, so this cannot select another
column's index. Ordinary indexes were not affected (BE binds them by column
unique ID). Covered by
`OlapTableTest.testInvertedIndexLookupIgnoresColumnNameCase` and a mixed-case
table in `test_variant_values_index_query`, which also runs the indexed query
with `enable_match_without_inverted_index=false`.
--
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]
