leonfin opened a new pull request, #7890:
URL: https://github.com/apache/geode/pull/7890

   * GEODE-10453 - in case of REMOVE_DUE_TO_GII_TOMBSTONE_CLEANUP and 
CompactRangeIndex, specify not to lookup old key, which is very expensive 
operation. It's actually broken and regression. All the tombstone entries are 
going to be NullToken and cause class cast exception for every single remove 
compare if looking up old key. There is no old key during initial tombstone 
image sync up from lead peer.
   
   ### For all changes:
   - [ GEODE-10453] Is there a JIRA ticket associated with this PR? Is it 
referenced in the commit message?
   
   - [Yes ] Has your PR been rebased against the latest commit within the 
target branch (typically `develop`)?
   
   - [ Yes] Is your initial contribution a single, squashed commit?
   
   - [ Yes] Does `gradlew build` run cleanly?
   
   - [ No] Have you written or updated unit tests to verify your changes?
   
   - [ No] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   
   <!-- Note:
   Please ensure that once the PR is submitted, check Concourse for build 
issues and
   submit an update to your PR as soon as possible. If you need help, please 
send an
   email to d...@geode.apache.org.
   -->
   


-- 
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

Reply via email to