albertogpz commented on a change in pull request #6028:
URL: https://github.com/apache/geode/pull/6028#discussion_r576391687
##########
File path:
geode-core/src/main/java/org/apache/geode/cache/query/internal/index/CompactRangeIndex.java
##########
@@ -1506,7 +1506,8 @@ private void applyProjectionForIndexInit(List
currentRuntimeIters)
: modifiedIndexExpr.evaluate(this.initContext);
if (indexKey == null) {
- indexKey = IndexManager.NULL;
+ // indexKey = IndexManager.NULL;
Review comment:
I think you are right. There is trouble with the != queries.
Shouldn't they not use the index?
According to the documentation, indexes are not used in expressions that
contain
NOT(https://geode.apache.org/docs/guide/19/developing/query_index/indexing_guidelines.html)
. I think the same could be done for queries using != which is very similar to
using NOT.
Nevertheless, when I checked the code, it seems that queries with NOT also
used the indexes.
----------------------------------------------------------------
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:
[email protected]
