Mailing-List: contact issues-help@geode.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@geode.apache.org
Date: Thu, 21 Sep 2017 16:22:01 +0000 (UTC)
From: "Fred Krone (JIRA)" <jira@apache.org>
To: issues@geode.apache.org
Message-ID: <JIRA.13059885.1490746889000.167786.1506010921091@Atlassian.JIRA>
In-Reply-To: <JIRA.13059885.1490746889000@Atlassian.JIRA>
References: <JIRA.13059885.1490746889000@Atlassian.JIRA> <JIRA.13059885.1490746889593@jira-lw-us.apache.org>
Subject: [jira] [Commented] (GEODE-2727) optimize new 1.8 ConcurrentMap
 default methods on Region
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Thu, 21 Sep 2017 16:22:07 -0000


    [ https://issues.apache.org/jira/browse/GEODE-2727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16175032#comment-16175032 ] 

Fred Krone commented on GEODE-2727:
-----------------------------------

[~dschneider] we should pick this up again


> optimize new 1.8 ConcurrentMap default methods on Region
> --------------------------------------------------------
>
>                 Key: GEODE-2727
>                 URL: https://issues.apache.org/jira/browse/GEODE-2727
>             Project: Geode
>          Issue Type: Improvement
>          Components: regions
>            Reporter: Darrel Schneider
>              Labels: region_interface, storage_2
>
> FRED: I can now edit this!
> In JDK 1.8 ConcurrentMap added the following default methods to the interface:
>   getOrDefault
>   computeIfAbsent
>   computeIfPresent
>   compute
>   merge
>   foreach
>   replaceAll
> All of the default implementations of these methods have issues with get returning null when the key actually exists. So they do not support invalid region entries.
> The other problem with all of them (except getOrDefault) is that they will make multiple round trips when done from a proxy. In a distributed environment we should implement these to send the lambda to the primary so that the entire operation can be done with one message and while the RegionEntry is locked.
> This would require that the lambda parameter by serializable which would be consistent with what we do for other parameters to Region methods (for example "put").
> From a performance point of view "foreach" and "replaceAll" are the worst since they bring back to whoever is executing the method all the keys and values from the entire cluster.
> It is also worth considering how the operations behave in a geode transaction.


--
This message was sent by Atlassian JIRA
(v6.4.14#64029)