[GEODE-9993] Redis SMOVE command should be atomic

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.15.0
• Fix Version/s: 1.15.0
• Component/s: redis
• Labels: blocks-1.15.0​ pull-request-available unreleased
• Assignee: Donal Evans
• Reporter: Donal Evans
• Created: 27/Jan/22 21:48
• Updated: 22/Jun/22 20:46
• Resolved: 01/Feb/22 20:47

Description

The documentation for the SMOVE command states that it is atomic. The current implementation in the geode-for-redis module is not, which could result in partially-applied moves if servers crash during a move.

The implementation should be changed to use the 'lockedExecuteInTransaction()' method in the SMoveExecutor class.

Issue Links

• links to: GitHub Pull Request #7321