ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dmitriy Setrakyan <dsetrak...@gridgain.com>
Subject Re: ClusterNodeLocalMap
Date Mon, 16 Feb 2015 07:09:21 GMT
Yakov, 

This seems like a simple change. Can you add it to the release?

Dmitriy



> On Feb 15, 2015, at 10:52 PM, Valentin Kulichenko <valentin.kulichenko@gmail.com>
wrote:
> 
> Agree.
> 
> On Sun, Feb 15, 2015 at 5:10 PM, Dmitriy Setrakyan <dsetrakyan@apache.org>
> wrote:
> 
>> Hi,
>> 
>> I just came across ClusterNodeLocalMap interface and cannot figure out why
>> we need to have it. It essentially extends java.util.ConcurrentMap and adds
>> 2 silly methods on top of it.
>> 
>> I propose to remove it and simply return ConcurrentMap instead, like so:
>> ---
>> public ConcurrentMap nodeLocalMap();
>> ---
>> 
>> Thoughts?
>> 
>> D.
>> 

Mime
View raw message