albertogpz commented on a change in pull request #6028:
URL: https://github.com/apache/geode/pull/6028#discussion_r579429969
##########
File path:
geode-core/src/main/java/org/apache/geode/cache/query/internal/CompiledIndexOperation.java
##########
@@ -104,7 +105,13 @@ public Object evaluate(ExecutionContext context) throws
TypeMismatchException,
}
if (rcvr instanceof Map) {
- return ((Map) rcvr).get(index);
+ if (context instanceof QueryExecutionContext) {
Review comment:
I have pushed a new commit with some test cases to compare the results
of queries with the indexes and without them.
As you can see, the tests when indexes are used fail.
The funny thing is that these test cases also fail if run against the latest
code in development. So I think there is something not working right with the
indexes, with or without my change.
----------------------------------------------------------------
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]
