[GEODE-10161] Clean up synchronization in RedisList

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.15.0
• Component/s: redis
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Jens Deppe
• Created: 24/Mar/22 13:43
• Updated: 22/Jun/22 20:46
• Resolved: 21/Apr/22 20:01

Description

Prior to adding versioning, we needed synchronized on various helper methods that modified the internal list data structure. This was in order to ensure exclusive access in the event of a toData() call (during GII/bucket movement). toData() is also synchronized. However, now that we're synchronizing within more of the 'top-level' methods in RedisList, (because we're also changing the version value), I think that we should be able to remove all of the synchronized modifiers on the smaller helper methods.

Issue Links

• links to: GitHub Pull Request #7579