[ https://issues.apache.org/jira/browse/GEODE-5135?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kirk Lund updated GEODE-5135: ----------------------------- Labels: AbstractRegionMap (was: ) > Refactor the static method AbstractRegionMap.createCBEvent > ---------------------------------------------------------- > > Key: GEODE-5135 > URL: https://issues.apache.org/jira/browse/GEODE-5135 > Project: Geode > Issue Type: Improvement > Components: transactions > Reporter: Darrel Schneider > Priority: Major > Labels: AbstractRegionMap > > AbstractRegionMap.createCBEvent would make more sense as a factory method on > EntryEventImpl. Note that ProxyRegionMap currently calls static methods on > AbstractRegionMap. > Since ProxyRegionMap and AbstractRegionMap both implement the RegionMap > interface it would be better to either have a common ancestor class for both > these classes that contains this common code they share or move this code to > methods on the RegionMap interface. It would be nice if ProxyRegionMap did > not depend on anything in AbstractRegionMap. > One idea would be to rename AbstractRegionMap to StorageRegionMap and then > have the common ancestor class that both ProxyRegionMap and StorageRegionMap > extend by named AbstractRegionMap. -- This message was sent by Atlassian JIRA (v7.6.3#76005)