[GEODE-5063] AbstractRegionMap.basicPut may get old value more than once

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: regions
• Labels: AbstractRegionMap
• Assignee: Unassigned
• Reporter: Darrel Schneider
• Created: 12/Apr/18 21:33
• Updated: 23/Apr/18 17:19

Description

AbstractRegionMap.doPutOnRegionEntry calls both of the following:

      setOldValueForDelta(putInfo);

      setOldValueInEvent(putInfo);

I think the logic in setOldValueForDelta can be moved into setOldValueInEvent.

As it is now we might fetch the old value twice when using delta.