leonfin commented on PR #7890: URL: https://github.com/apache/geode/pull/7890#issuecomment-3229653467
BTW original author even said in the original commit https://bitbucket.tools.tsimagine.com/projects/CORE/repos/geode/commits/9e7db8602df299af57d2ea824ba656fd24d54600#geode-core/src/main/java/org/apache/geode/cache/query/internal/index/CompactRangeIndex.java "... // we also know that a regular remove won't work due to the entry no longer being present // We know the old key so let's just remove mapping from the old key" Not sure why down the code path it was missed that for GII (get initial image), the findOldKey was defaulted to true. We're already using this patched version in prod for 6+ months. -- 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: notifications-unsubscr...@geode.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
