[GEODE-2860] refactor EventTracker to be on DistributedRegion instead of LocalRegion

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: regions
• Labels: storage_3
• Assignee: Nick Reich
• Reporter: Darrel Schneider
• Created: 02/May/17 18:04
• Updated: 20/Apr/18 19:51
• Resolved: 18/Jul/17 15:18

Description

Currently LocalRegion has a non-final field named "eventTracker". It is initialized in a method named createEventTracker which does nothing on LocalRegion but is implemented on DistributedRegion and BucketRegion to initialize the eventTracker field.

I think things would be clearer if this field was moved to DistributedRegion.
All the code on LocalRegion that currently tests for a non-null eventTracker can be changed to do nothing and overridden on DistributedRegion to use its eventTracker. DistributedRegion can make this field final and always set it in its constructor. Since BucketRegion extends DistributedRegion it does not to do anything (it currently implements createEventTracker but that was not needed since it inherits the same impl from DistributedRegion).

Issue Links

• links to: GitHub Pull Request #616