pivotal-jbarrett commented on a change in pull request #7262:
URL: https://github.com/apache/geode/pull/7262#discussion_r819191456
##########
File path:
geode-core/src/main/java/org/apache/geode/cache/query/internal/index/AbstractIndex.java
##########
@@ -283,9 +284,11 @@ public void query(Object key, int operator, Collection
results, ExecutionContext
}
@Override
- public void query(Object key, int operator, Collection results, @Retained
CompiledValue iterOp,
- RuntimeIterator indpndntItr, ExecutionContext context, List projAttrib,
- SelectResults intermediateResults, boolean isIntersection) throws
TypeMismatchException,
+ public void query(Object key, int operator, Collection<Object> results,
+ @Retained CompiledValue iterOp,
+ RuntimeIterator runtimeIterator, ExecutionContext context, List<?>
projAttrib,
Review comment:
My guess is that this was change as more of a cascading replacement. I
didn't look to correct the poorly named symbols in all files I touched, just
some of the more ugly ones in areas I was doing heavier work in.
--
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]