[
https://issues.apache.org/jira/browse/GEODE-6704?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Darrel Schneider updated GEODE-6704:
------------------------------------
Priority: Minor (was: Major)
> AbstractRegion.allGatewaySenderIds can be improved
> --------------------------------------------------
>
> Key: GEODE-6704
> URL: https://issues.apache.org/jira/browse/GEODE-6704
> Project: Geode
> Issue Type: Improvement
> Components: core
> Reporter: Darrel Schneider
> Priority: Minor
>
> Two things I noticed about AbstractRegion.allGatewaySenderIds that can be
> easily improved:
> # It is currently implemented with CopyOnWriteArraySet but is never modified
> in place and is searched. These searches end up doing a linear search since
> the underlying impl is an ArrayList.
> # The gettor method for allGatewaySenderIds always creates an unmodifiable
> wrapper. This is just garbage since the set is never modified.
> A prototype fix is in: 9d978c615d4e6ccfc3d1ab731cf2e76bb59d1fec
> The settor method now uses a HashSet and makes sure that it is always wrapped
> in an unmodifiable wrapper. This allows the gettor to just return the
> reference.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
