github-actions[bot] commented on code in PR #69761:
URL: https://github.com/apache/doris/pull/69761#discussion_r4432330113
##########
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:
[P1] Match values indexes case-insensitively
CREATE TABLE accepts index column identifiers case-insensitively, but
`translateToCatalogStyle()` keeps the original `cols` spelling and the new
`getInvertedIndexes()` compares it with `column.getName()` using `equals`. Thus
a valid table with `V VARIANT` and `INDEX values_idx(v) ...
variant_index_scope=values` persists an index that
`hasVariantValuesIndex()`/`getVariantValuesIndex()` cannot see. `V MATCH_ANY
...` is then treated as a non-values root, so the declared index is not used
(and with residual fallback disabled the query fails). Compare names
case-insensitively or persist canonical column names, and add a mixed-case
root-MATCH regression.
--
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]
