[GEODE-6058] recordVersion should allow update higher local version if for non-persistent region

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.9.0
• Labels: pull-request-available swat
• Assignee: Xiaojian Zhou
• Reporter: Xiaojian Zhou
• Created: 14/Nov/18 22:53
• Updated: 24/Apr/19 22:22
• Resolved: 15/Nov/18 22:53

Description

When we introduced RVV, we put an assertion to disallow recordVersion to be a higher local version. i.e. the local version should be increased by local member, not from remote (such as GII).

 

But we forgot to enforce this assertion for persistent region only.

 

The reason is: If in persist region, the operation increased local version and persist it first before sending out distribution. And it will recover from disk. So there's no reason for a persistent region to get a higher local version from remote (such as GII). 

 

But this is not the case for non-persistent region. 

 

If we have a non-persistent region with concurrencyCheckEnabled, memberA initiated an operation and distributed to remote. The remote member keeps memberA's region version as 1. If we destroy and recreate region on memberA, it cannot recover from disk. The only way to get RVV back is through GII from remote. So we have to allow updating region version 1 for memberA into memberA's empty RVV. 

Issue Links

• links to: GitHub Pull Request #2851