Re: Remove GridBoundedConcurrentLinkedHashMap and replace its usages with ConcurrentLinkedHashMap

[Yakov Zhdanov]

This ticket is more about shaping up the code and will not have any visible
effect on the system. These two classes are very similar and I doubt if
inheritance that has been introduced there adds any value since
ConcurrentLinkedHashMap already has maxSize property. Anyway, it should be
a pretty simple change to replace about 10 usages
with ConcurrentLinkedHashMap.

Konstantin, this one looks more interesting to me -
https://issues.apache.org/jira/browse/IGNITE-2655

--Yakov

2016-02-15 19:34 GMT+03:00 Denis Magda :

>
> Hi Konstantin,
>
> I've shared my thoughts on your questions. Yakov, as a creator of the
> ticket, please share your view on whether these changes are really needed
> or not.
>
> --
> Denis
>
>  Forwarded Message 
> Subject: Re: Hello
> Date: Mon, 15 Feb 2016 15:08:25 +0300
> From: Konstantin Margorin  
> Reply-To: dev@ignite.apache.org
> To: dev@ignite.apache.org
>
> Hi all.
>
> Could please someone review my comment 
> tohttps://issues.apache.org/jira/browse/IGNITE-1783
>
> It's easy to make a patch, but this will lead to longer arguments list in
> some places. Is it ok?
>
> On Tue, Feb 9, 2016 at 3:11 PM, Vladimir Ozerov  
> 
> wrote:
>
> > Konstantin,
> >
> > This ticket is more about build process. You can also look at the following
> > ones:
> > https://issues.apache.org/jira/browse/IGNITE-1783
> > https://issues.apache.org/jira/browse/IGNITE-2555
> >
> > Vladimir.
> >
> > On Tue, Feb 9, 2016 at 2:57 PM, Konstantin Margorin  
> > 
> > wrote:
> >
> > > Hi Vladimir.
> > >
> > > I had searched through open tickets for newbies. And looks like the only
> > > open unassigned ticked is:
> > > https://issues.apache.org/jira/browse/IGNITE-2049
> > >
> > > Is this ticket right point to start?
> > >
> > > Best regards,
> > > Konstantin
> > >
> > > On Tue, Feb 9, 2016 at 2:30 PM, Vladimir Ozerov  
> > > 
> > > wrote:
> > >
> > > > Konstantin, sorry I confused the name.
> > > >
> > > > Should you have any further questions please feel free to ask.
> > > >
> > > > On Tue, Feb 9, 2016 at 2:29 PM, Vladimir Ozerov  
> > > > 
> > > > wrote:
> > > >
> > > > > Hi Carlos,
> > > > >
> > > > > Welcome to the Ignite community!
> > > > >
> > > > > You should get familiar with Ignite development process described
> > here:
> > > > >
> > https://cwiki.apache.org/confluence/display/IGNITE/Development+Process
> > > > >
> > > > > Instructions on how to contribute can be found here:
> > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute
> > > > >
> > > > > Once you got familiar and were able to run a few examples, you should
> > > > pick
> > > > > a Jira ticket you would like to start on. Send an email to the dev
> > > list,
> > > > so
> > > > > we can add you as a contributor in Jira.
> > > > >
> > > > > You can find interesting tickets to get started here:
> > > > > https://ignite.apache.org/community/contribute.html#pick-ticket
> > > > >
> > > > > Looking forward to your contributions!
> > > > >
> > > > > Regards,
> > > > > Vladimir.
> > > > >
> > > > >
> > > > > On Tue, Feb 9, 2016 at 12:58 PM, Konstantin Margorin <
> > rus...@gmail.com
> > > >
> > > > > wrote:
> > > > >
> > > > >> Hi All,
> > > > >>
> > > > >> I'm Konstantin Margorin from Anturis.
> > > > >>
> > > > >> I would like to contribute my effort on Apache Ignite project and
> > > > >> appreciate if I could join with you.
> > > > >>
> > > > >> Looking forward to your reply.
> > > > >>
> > > > >> Best regards,
> > > > >> Konstantin
> > > > >>
> > > > >
> > > > >
> > > >
> > >
> >
>
>
>
>
>

Remove GridBoundedConcurrentLinkedHashMap and replace its usages with ConcurrentLinkedHashMap

[Denis Magda]

Hi Konstantin,

I've shared my thoughts on your questions. Yakov, as a creator of the 
ticket, please share your view on whether these changes are really 
needed or not.


--
Denis

 Forwarded Message 
Subject:Re: Hello
Date:   Mon, 15 Feb 2016 15:08:25 +0300
From:   Konstantin Margorin 
Reply-To:   dev@ignite.apache.org
To: dev@ignite.apache.org



Hi all.

Could please someone review my comment to
https://issues.apache.org/jira/browse/IGNITE-1783

It's easy to make a patch, but this will lead to longer arguments list in
some places. Is it ok?

On Tue, Feb 9, 2016 at 3:11 PM, Vladimir Ozerov 
wrote:


Konstantin,

This ticket is more about build process. You can also look at the following
ones:
https://issues.apache.org/jira/browse/IGNITE-1783
https://issues.apache.org/jira/browse/IGNITE-2555

Vladimir.

On Tue, Feb 9, 2016 at 2:57 PM, Konstantin Margorin 
wrote:

> Hi Vladimir.
>
> I had searched through open tickets for newbies. And looks like the only
> open unassigned ticked is:
> https://issues.apache.org/jira/browse/IGNITE-2049
>
> Is this ticket right point to start?
>
> Best regards,
> Konstantin
>
> On Tue, Feb 9, 2016 at 2:30 PM, Vladimir Ozerov 
> wrote:
>
> > Konstantin, sorry I confused the name.
> >
> > Should you have any further questions please feel free to ask.
> >
> > On Tue, Feb 9, 2016 at 2:29 PM, Vladimir Ozerov 
> > wrote:
> >
> > > Hi Carlos,
> > >
> > > Welcome to the Ignite community!
> > >
> > > You should get familiar with Ignite development process described
here:
> > >
https://cwiki.apache.org/confluence/display/IGNITE/Development+Process
> > >
> > > Instructions on how to contribute can be found here:
> > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute
> > >
> > > Once you got familiar and were able to run a few examples, you should
> > pick
> > > a Jira ticket you would like to start on. Send an email to the dev
> list,
> > so
> > > we can add you as a contributor in Jira.
> > >
> > > You can find interesting tickets to get started here:
> > > https://ignite.apache.org/community/contribute.html#pick-ticket
> > >
> > > Looking forward to your contributions!
> > >
> > > Regards,
> > > Vladimir.
> > >
> > >
> > > On Tue, Feb 9, 2016 at 12:58 PM, Konstantin Margorin <
rus...@gmail.com
> >
> > > wrote:
> > >
> > >> Hi All,
> > >>
> > >> I'm Konstantin Margorin from Anturis.
> > >>
> > >> I would like to contribute my effort on Apache Ignite project and
> > >> appreciate if I could join with you.
> > >>
> > >> Looking forward to your reply.
> > >>
> > >> Best regards,
> > >> Konstantin
> > >>
> > >
> > >
> >
>

[jira] [Created] (IGNITE-1783) Remove GridBoundedConcurrentLinkedHashMap and replace its usages with ConcurrentLinkedHashMap

[Yakov Zhdanov (JIRA)]

Yakov Zhdanov created IGNITE-1783:
-

 Summary: Remove GridBoundedConcurrentLinkedHashMap and replace its 
usages with ConcurrentLinkedHashMap
 Key: IGNITE-1783
 URL: https://issues.apache.org/jira/browse/IGNITE-1783
 Project: Ignite
  Issue Type: Task
Reporter: Yakov Zhdanov
Priority: Trivial


>From what I see {{GridBoundedConcurrentLinkedHashMap}} is redundant and the 
>only difference from {{ConcurrentLinkedHashMap}} is in order of constructor 
>arguments.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)