[GEODE-6899] retried client should set last try's version tag if found

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.10.0
• Assignee: Unassigned
• Reporter: Xiaojian Zhou
• Created: 22/Jun/19 00:09
• Updated: 26/Sep/19 18:04
• Resolved: 27/Jun/19 16:52

Description

client does a put to serverA with replicated region, serverA distributed to B and C, before the distribution arrived at C, A is killed. Then client could retry to C. C noticed this is a retry operation, it will search for previous try's version tag.

The found tag should be set into the current event. Interesting thing is:
I found other operations, such as PutIfAbsent and create, they both did that. But replace (i.e. put) did not.

Another issue is: GEODE-6802 introduce a synchronizeIfNotScheduled(). But there could be a race that membershipListener is also scheduling. The fix is to pause 1 second before calling the newly introduced synchronizeIfNotScheduled().

Issue Links

• links to: GitHub Pull Request #3743