----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50615/ -----------------------------------------------------------
Review request for geode, anilkumar gingade and nabarun nag. Repository: geode Description ------- The keysToRemove set was being removed from during iteration of the index. This is as expected, however a copy of the keysToRemove should be made first because the same set will be passed to the next bucket. We would end up removing from the original set and as we continued the query to the next bucket, it would eventually be an empty set. PdxString and Strings were not being properly compared in TypeUtils (which is used by CompactRangeIndexes, among others) Diffs ----- geode-core/src/main/java/com/gemstone/gemfire/cache/query/internal/index/MemoryIndexStore.java 83f8c02 geode-core/src/main/java/com/gemstone/gemfire/cache/query/internal/types/TypeUtils.java 14c798f geode-core/src/test/java/com/gemstone/gemfire/cache/query/dunit/CompactRangeIndexQueryDUnitTest.java PRE-CREATION geode-core/src/test/java/com/gemstone/gemfire/cache/query/internal/types/TypeUtilTest.java PRE-CREATION Diff: https://reviews.apache.org/r/50615/diff/ Testing ------- Thanks, Jason Huynh
