You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by "Gary D. Gregory" <gg...@apache.org> on 2024/09/12 16:58:58 UTC
[beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Hi All,
For the upcoming 2.0.0-M1, I plan on replacing the custom WeakFastHashMap with the stock ConcurrentHashMap.
If you think this is a bad idea, please tell us why.
Gary
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
RE: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Josh Fenlason <Jo...@veritas.com.INVALID>.
Gary,
I appreciate all the work you put into the various Apache Commons projects, so I hope this does not sound ungrateful. I would like to ask what can be done to increase the urgency for releasing BeanUtils 2.0. Due to requirements from our public sector customers, we need be STIG compliant (https://www.stigviewer.com/controls/800-53/SA-22) and the age of the latest BeanUtil release is impeding that. Based on the activity on the mailing list it appears that there are several others that are in a similar situation. I know that you have been doing some work in this regard and already do a lot for other Apache Commons projects as well, so I am not coming and asking you to handle this all on your own. There are several in the community who are offering to help get BeanUtils 2 out the door. I would like to respectfully ask what I can be doing to hasten the new release by the end of October at the latest so I can avoid having to do my own fork of the library.
Thanks,
Josh.
-----Original Message-----
From: Josh Fenlason <Jo...@veritas.com.INVALID>
Sent: Wednesday, September 25, 2024 3:25 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: RE: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
Sounds good. I'll keep an eye out for it.
-----Original Message-----
From: Gary Gregory <ga...@gmail.com>
Sent: Wednesday, September 25, 2024 3:22 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
I'll ping this thread when there is something in git master to test in more depth.
Gary
On Wed, Sep 25, 2024 at 4:01 PM Josh Fenlason <Jo...@veritas.com.invalid> wrote:
>
> Excellent, thanks for that encouraging update! Is there anything I can assist with?
> Josh.
>
> -----Original Message-----
> From: Gary Gregory <ga...@gmail.com>
> Sent: Wednesday, September 25, 2024 2:52 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
>
> CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
>
>
>
> FWIW, I have a patch in progress using the Apache Grooy's version of the class, so licensing will not be an issue, Apache to Apache.
>
> If you look at the recent commits in git master, you'll see that I've started adding missing tests we need before we add anything else...
>
> I also have tests written that fit in the existing test framework.
> These will likely need to be beefed up. I saw zero tests for this class in the Groovy code base.
>
> Stay tuned.
>
> Gary
>
> On Wed, Sep 25, 2024 at 3:37 PM Josh Fenlason <Jo...@veritas.com.invalid> wrote:
> >
> > Are legal concerns of using Netty's ConcurrentWeakKeyHashMap the only issue with Melloware's PR (https://github.com/apache/commons-beanutils/pull/56) from a couple of years ago? That PR mentions http://creativecommons.org/licenses/publicdomain for details on the license, which unfortunately no longer works. However, looking at the Netty source (https://github.com/netty/netty/blob/3.9/src/main/java/org/jboss/netty/util/internal/ConcurrentWeakKeyHashMap.java), they have an Apache 2 license. Doesn't that alleviate any license concerns? Can we resuscitate Melloware's PR and wrap up this issue?
> > Thanks,
> > Josh.
> >
> > -----Original Message-----
> > From: Gary Gregory <ga...@gmail.com>
> > Sent: Monday, September 16, 2024 4:26 PM
> > To: Commons Developers List <de...@commons.apache.org>
> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > ConcurrentHashMap
> >
> >
> > CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
> >
> >
> >
> > Which points to:
> > https://issu/
> > es.apache.org%2Fjira%2Fbrowse%2FBEANUTILS-509&data=05%7C02%7CJosh.Fe
> > nl
> > ason%40veritas.com%7C594b7769bda54bbd892d08dcdd9ba40c%7Cfc8e13c0422c
> > 4c
> > 55b3eaca318e6cac32%7C0%7C0%7C638628907749754165%7CUnknown%7CTWFpbGZs
> > b3
> > d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D
> > %7
> > C0%7C%7C%7C&sdata=SGR%2B0SfgS%2F7gmd8VCPOlTmmlwQQNNp77J05QI5fFaBc%3D
> > &r
> > eserved=0
> >
> > Gary
> >
> > On Mon, Sep 16, 2024, 5:24 PM Gary Gregory <ga...@gmail.com> wrote:
> >
> > > Related:
> > > https://issu/
> > > es.apache.org%2Fjira%2Fbrowse%2FCOLLECTIONS-700&data=05%7C02%7CJosh.
> > > Fe
> > > nlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c0
> > > 42
> > > 2c
> > > 4c55b3eaca318e6cac32%7C0%7C0%7C638621188049710140%7CUnknown%7CTWFp
> > > bG
> > > Zs
> > > b3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn
> > > 0%
> > > 3D
> > > %7C0%7C%7C%7C&sdata=UspxFjiRSi4rTJSW7wNHs2XESjZ0Cvbm350jrmVMD3I%3D
> > > &r
> > > es
> > > erved=0
> > >
> > > Gary
> > >
> > > On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
> > >
> > >> IMO I think it ok...so Gary which part do you feel unsafe in it
> > >> ？I
> > >> must admit I am not that familiar to lawyer things..
> > >> ________________________________
> > >> From: Melloware Inc <me...@gmail.com>
> > >> Sent: Saturday, September 14, 2024 9:18:08 PM
> > >> To: Commons Developers List <de...@commons.apache.org>
> > >> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> ConcurrentHashMap
> > >>
> > >> Well in my PR the license in the file says this.
> > >>
> > >> Written by Doug Lea with assistance from members of JCP JSR-166
> > >> Expert Group and released to the public domain, as explained at
> > >> @see <a
> > >> href="https://nam12.safelinks.protection.outlook.com/?url=http%3A
> > >> %252
> > >> F%25
> > >> 2Fcreativecommons.org%2Flicenses%2Fpublicdomain&data=05%7C02%7CJo
> > >> sh
> > >> .F
> > >> enlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13
> > >> c0
> > >> 42
> > >> 2c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049723177%7CUnknown%7CT
> > >> WF
> > >> pb
> > >> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVC
> > >> I6
> > >> Mn
> > >> 0%3D%7C0%7C%7C%7C&sdata=Vgo5BT3zXWMHgn4iv1PKrw0kZd2JMUo2V1o78%2FM
> > >> uF
> > >> po
> > >> %3D&reserved=0">Creative
> > >> Commons</a>
> > >>
> > >> it says its public domain.
> > >>
> > >> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
> > >>
> > >> > why not write an email to lea ..maybe he be so kind that would
> > >> > be glad
> > >> to
> > >> > offer one mit-like license copy of that class
> > >> > ________________________________
> > >> > From: Melloware Inc <me...@gmail.com>
> > >> > Sent: Saturday, September 14, 2024 8:32:38 PM
> > >> > To: Commons Developers List <de...@commons.apache.org>
> > >> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> > ConcurrentHashMap
> > >> >
> > >> > I submitted this PR years ago using ConcurrentWeakKeyHashMap
> > >> > from Doug
> > >> Lea
> > >> > and the Netty team but Gary had concerns from legal about being
> > >> > able to
> > >> use
> > >> > it.
> > >> >
> > >> > PR is here:
> > >> > https://g/
> > >> > ithub.com%2Fapache%2Fcommons-beanutils%2Fpull%2F56&data=05%7C02
> > >> > %7
> > >> > CJ
> > >> > osh.Fenlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7
> > >> > Cf
> > >> > c8
> > >> > e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049734927%7CUn
> > >> > kn
> > >> > ow
> > >> > n%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1
> > >> > ha
> > >> > Ww
> > >> > iLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=ERcuJKHJW%2F7plLn9Pj5nUaA4rF
> > >> > Cu
> > >> > kB
> > >> > s8V6WP%2BABCzRc%3D&reserved=0
> > >> >
> > >> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> > >> >
> > >> > > I thought old d-lea bro have a concurrent weak hashmap
> > >> > > implement somewhere...why not just ask him after all nobody
> > >> > > here more expert than him in this area
> > >> > > ________________________________
> > >> > > From: Niall Pemberton <ni...@gmail.com>
> > >> > > Sent: Saturday, September 14, 2024 1:46:32 PM
> > >> > > To: Commons Developers List <de...@commons.apache.org>
> > >> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> > > ConcurrentHashMap
> > >> > >
> > >> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory
> > >> > > <gg...@apache.org>
> > >> > wrote:
> > >> > >
> > >> > > > Hi All,
> > >> > > >
> > >> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> > >> > WeakFastHashMap
> > >> > > > with the stock ConcurrentHashMap.
> > >> > > >
> > >> > > > If you think this is a bad idea, please tell us why.
> > >> > >
> > >> > >
> > >> > > It’s a good idea for the “fast” part, but the “weak” aspect
> > >> > > also
> > >> needs to
> > >> > > be retained - so in principle yes, but the implementation
> > >> > > detail
> > >> matters.
> > >> > >
> > >> > > BeanUtils caused memory issues in multiple ClassLoader
> > >> > > environments
> > >> (e.g.
> > >> > > WebApp containers) because of strong references to classes
> > >> > > prevented
> > >> them
> > >> > > from being garbage collected if the app was reloaded/restarted.
> > >> > > So
> > >> this
> > >> > > weak map was introduced to resolve that issue.
> > >> > >
> > >> > > Niall
> > >> > >
> > >> > >
> > >> > > >
> > >> > > > Gary
> > >> > > >
> > >> > > >
> > >> -----------------------------------------------------------------
> > >> --
> > >> --
> > >> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > >> > > > For additional commands, e-mail:
> > >> > > > dev-help@commons.apache.org
> > >> > > >
> > >> > > >
> > >> > >
> > >> >
> > >> >
> > >> > --
> > >> > ==============================
> > >> > Melloware
> > >> > mellowaredev@gmail.com
> > >> > http://me/
> > >> > lloware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17
> > >> > c9
> > >> > a6
> > >> > a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C
> > >> > 0%
> > >> > 7C
> > >> > 0%7C638621188049744703%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAw
> > >> > MD
> > >> > Ai
> > >> > LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sda
> > >> > ta
> > >> > =o
> > >> > 6%2BhNto8to7f4E1GZtN%2BpFVpoFx0b4gI4Exy90G7apk%3D&reserved=0
> > >> > ==============================
> > >> >
> > >>
> > >>
> > >> --
> > >> ==============================
> > >> Melloware
> > >> mellowaredev@gmail.com
> > >> http://mell/
> > >> oware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6
> > >> a1
> > >> 3a
> > >> 540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%
> > >> 7C
> > >> 63
> > >> 8621188049751739%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQ
> > >> Ij
> > >> oi
> > >> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=Iof1UJp
> > >> pc
> > >> Gh
> > >> PV%2F5T2Lmo5i%2BX1nA%2F0V1M7eyEqjvHtUQ%3D&reserved=0
> > >> ==============================
> > >>
> > >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
B KKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKCB [ X ܚX KK[XZ[
] ][ X ܚX P [[ۜ˘\X K ܙ B ܈Y][ۘ[ [X[ K[XZ[
] Z[ [[ۜ˘\X K ܙ B
RE: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Josh Fenlason <Jo...@veritas.com.INVALID>.
Sounds good. I'll keep an eye out for it.
-----Original Message-----
From: Gary Gregory <ga...@gmail.com>
Sent: Wednesday, September 25, 2024 3:22 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
I'll ping this thread when there is something in git master to test in more depth.
Gary
On Wed, Sep 25, 2024 at 4:01 PM Josh Fenlason <Jo...@veritas.com.invalid> wrote:
>
> Excellent, thanks for that encouraging update! Is there anything I can assist with?
> Josh.
>
> -----Original Message-----
> From: Gary Gregory <ga...@gmail.com>
> Sent: Wednesday, September 25, 2024 2:52 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
>
> CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
>
>
>
> FWIW, I have a patch in progress using the Apache Grooy's version of the class, so licensing will not be an issue, Apache to Apache.
>
> If you look at the recent commits in git master, you'll see that I've started adding missing tests we need before we add anything else...
>
> I also have tests written that fit in the existing test framework.
> These will likely need to be beefed up. I saw zero tests for this class in the Groovy code base.
>
> Stay tuned.
>
> Gary
>
> On Wed, Sep 25, 2024 at 3:37 PM Josh Fenlason <Jo...@veritas.com.invalid> wrote:
> >
> > Are legal concerns of using Netty's ConcurrentWeakKeyHashMap the only issue with Melloware's PR (https://github.com/apache/commons-beanutils/pull/56) from a couple of years ago? That PR mentions http://creativecommons.org/licenses/publicdomain for details on the license, which unfortunately no longer works. However, looking at the Netty source (https://github.com/netty/netty/blob/3.9/src/main/java/org/jboss/netty/util/internal/ConcurrentWeakKeyHashMap.java), they have an Apache 2 license. Doesn't that alleviate any license concerns? Can we resuscitate Melloware's PR and wrap up this issue?
> > Thanks,
> > Josh.
> >
> > -----Original Message-----
> > From: Gary Gregory <ga...@gmail.com>
> > Sent: Monday, September 16, 2024 4:26 PM
> > To: Commons Developers List <de...@commons.apache.org>
> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > ConcurrentHashMap
> >
> >
> > CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
> >
> >
> >
> > Which points to:
> > https://issu/
> > es.apache.org%2Fjira%2Fbrowse%2FBEANUTILS-509&data=05%7C02%7CJosh.Fe
> > nl
> > ason%40veritas.com%7C594b7769bda54bbd892d08dcdd9ba40c%7Cfc8e13c0422c
> > 4c
> > 55b3eaca318e6cac32%7C0%7C0%7C638628907749754165%7CUnknown%7CTWFpbGZs
> > b3
> > d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D
> > %7
> > C0%7C%7C%7C&sdata=SGR%2B0SfgS%2F7gmd8VCPOlTmmlwQQNNp77J05QI5fFaBc%3D
> > &r
> > eserved=0
> >
> > Gary
> >
> > On Mon, Sep 16, 2024, 5:24 PM Gary Gregory <ga...@gmail.com> wrote:
> >
> > > Related:
> > > https://issu/
> > > es.apache.org%2Fjira%2Fbrowse%2FCOLLECTIONS-700&data=05%7C02%7CJosh.
> > > Fe
> > > nlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c0
> > > 42
> > > 2c
> > > 4c55b3eaca318e6cac32%7C0%7C0%7C638621188049710140%7CUnknown%7CTWFp
> > > bG
> > > Zs
> > > b3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn
> > > 0%
> > > 3D
> > > %7C0%7C%7C%7C&sdata=UspxFjiRSi4rTJSW7wNHs2XESjZ0Cvbm350jrmVMD3I%3D
> > > &r
> > > es
> > > erved=0
> > >
> > > Gary
> > >
> > > On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
> > >
> > >> IMO I think it ok...so Gary which part do you feel unsafe in it
> > >> ？I
> > >> must admit I am not that familiar to lawyer things..
> > >> ________________________________
> > >> From: Melloware Inc <me...@gmail.com>
> > >> Sent: Saturday, September 14, 2024 9:18:08 PM
> > >> To: Commons Developers List <de...@commons.apache.org>
> > >> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> ConcurrentHashMap
> > >>
> > >> Well in my PR the license in the file says this.
> > >>
> > >> Written by Doug Lea with assistance from members of JCP JSR-166
> > >> Expert Group and released to the public domain, as explained at
> > >> @see <a
> > >> href="https://nam12.safelinks.protection.outlook.com/?url=http%3A
> > >> %252
> > >> F%25
> > >> 2Fcreativecommons.org%2Flicenses%2Fpublicdomain&data=05%7C02%7CJo
> > >> sh
> > >> .F
> > >> enlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13
> > >> c0
> > >> 42
> > >> 2c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049723177%7CUnknown%7CT
> > >> WF
> > >> pb
> > >> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVC
> > >> I6
> > >> Mn
> > >> 0%3D%7C0%7C%7C%7C&sdata=Vgo5BT3zXWMHgn4iv1PKrw0kZd2JMUo2V1o78%2FM
> > >> uF
> > >> po
> > >> %3D&reserved=0">Creative
> > >> Commons</a>
> > >>
> > >> it says its public domain.
> > >>
> > >> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
> > >>
> > >> > why not write an email to lea ..maybe he be so kind that would
> > >> > be glad
> > >> to
> > >> > offer one mit-like license copy of that class
> > >> > ________________________________
> > >> > From: Melloware Inc <me...@gmail.com>
> > >> > Sent: Saturday, September 14, 2024 8:32:38 PM
> > >> > To: Commons Developers List <de...@commons.apache.org>
> > >> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> > ConcurrentHashMap
> > >> >
> > >> > I submitted this PR years ago using ConcurrentWeakKeyHashMap
> > >> > from Doug
> > >> Lea
> > >> > and the Netty team but Gary had concerns from legal about being
> > >> > able to
> > >> use
> > >> > it.
> > >> >
> > >> > PR is here:
> > >> > https://g/
> > >> > ithub.com%2Fapache%2Fcommons-beanutils%2Fpull%2F56&data=05%7C02
> > >> > %7
> > >> > CJ
> > >> > osh.Fenlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7
> > >> > Cf
> > >> > c8
> > >> > e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049734927%7CUn
> > >> > kn
> > >> > ow
> > >> > n%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1
> > >> > ha
> > >> > Ww
> > >> > iLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=ERcuJKHJW%2F7plLn9Pj5nUaA4rF
> > >> > Cu
> > >> > kB
> > >> > s8V6WP%2BABCzRc%3D&reserved=0
> > >> >
> > >> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> > >> >
> > >> > > I thought old d-lea bro have a concurrent weak hashmap
> > >> > > implement somewhere...why not just ask him after all nobody
> > >> > > here more expert than him in this area
> > >> > > ________________________________
> > >> > > From: Niall Pemberton <ni...@gmail.com>
> > >> > > Sent: Saturday, September 14, 2024 1:46:32 PM
> > >> > > To: Commons Developers List <de...@commons.apache.org>
> > >> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> > > ConcurrentHashMap
> > >> > >
> > >> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory
> > >> > > <gg...@apache.org>
> > >> > wrote:
> > >> > >
> > >> > > > Hi All,
> > >> > > >
> > >> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> > >> > WeakFastHashMap
> > >> > > > with the stock ConcurrentHashMap.
> > >> > > >
> > >> > > > If you think this is a bad idea, please tell us why.
> > >> > >
> > >> > >
> > >> > > It’s a good idea for the “fast” part, but the “weak” aspect
> > >> > > also
> > >> needs to
> > >> > > be retained - so in principle yes, but the implementation
> > >> > > detail
> > >> matters.
> > >> > >
> > >> > > BeanUtils caused memory issues in multiple ClassLoader
> > >> > > environments
> > >> (e.g.
> > >> > > WebApp containers) because of strong references to classes
> > >> > > prevented
> > >> them
> > >> > > from being garbage collected if the app was reloaded/restarted.
> > >> > > So
> > >> this
> > >> > > weak map was introduced to resolve that issue.
> > >> > >
> > >> > > Niall
> > >> > >
> > >> > >
> > >> > > >
> > >> > > > Gary
> > >> > > >
> > >> > > >
> > >> -----------------------------------------------------------------
> > >> --
> > >> --
> > >> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > >> > > > For additional commands, e-mail:
> > >> > > > dev-help@commons.apache.org
> > >> > > >
> > >> > > >
> > >> > >
> > >> >
> > >> >
> > >> > --
> > >> > ==============================
> > >> > Melloware
> > >> > mellowaredev@gmail.com
> > >> > http://me/
> > >> > lloware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17
> > >> > c9
> > >> > a6
> > >> > a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C
> > >> > 0%
> > >> > 7C
> > >> > 0%7C638621188049744703%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAw
> > >> > MD
> > >> > Ai
> > >> > LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sda
> > >> > ta
> > >> > =o
> > >> > 6%2BhNto8to7f4E1GZtN%2BpFVpoFx0b4gI4Exy90G7apk%3D&reserved=0
> > >> > ==============================
> > >> >
> > >>
> > >>
> > >> --
> > >> ==============================
> > >> Melloware
> > >> mellowaredev@gmail.com
> > >> http://mell/
> > >> oware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6
> > >> a1
> > >> 3a
> > >> 540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%
> > >> 7C
> > >> 63
> > >> 8621188049751739%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQ
> > >> Ij
> > >> oi
> > >> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=Iof1UJp
> > >> pc
> > >> Gh
> > >> PV%2F5T2Lmo5i%2BX1nA%2F0V1M7eyEqjvHtUQ%3D&reserved=0
> > >> ==============================
> > >>
> > >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Gary Gregory <ga...@gmail.com>.
I'll ping this thread when there is something in git master to test in
more depth.
Gary
On Wed, Sep 25, 2024 at 4:01 PM Josh Fenlason
<Jo...@veritas.com.invalid> wrote:
>
> Excellent, thanks for that encouraging update! Is there anything I can assist with?
> Josh.
>
> -----Original Message-----
> From: Gary Gregory <ga...@gmail.com>
> Sent: Wednesday, September 25, 2024 2:52 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
>
> CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
>
>
>
> FWIW, I have a patch in progress using the Apache Grooy's version of the class, so licensing will not be an issue, Apache to Apache.
>
> If you look at the recent commits in git master, you'll see that I've started adding missing tests we need before we add anything else...
>
> I also have tests written that fit in the existing test framework.
> These will likely need to be beefed up. I saw zero tests for this class in the Groovy code base.
>
> Stay tuned.
>
> Gary
>
> On Wed, Sep 25, 2024 at 3:37 PM Josh Fenlason <Jo...@veritas.com.invalid> wrote:
> >
> > Are legal concerns of using Netty's ConcurrentWeakKeyHashMap the only issue with Melloware's PR (https://github.com/apache/commons-beanutils/pull/56) from a couple of years ago? That PR mentions http://creativecommons.org/licenses/publicdomain for details on the license, which unfortunately no longer works. However, looking at the Netty source (https://github.com/netty/netty/blob/3.9/src/main/java/org/jboss/netty/util/internal/ConcurrentWeakKeyHashMap.java), they have an Apache 2 license. Doesn't that alleviate any license concerns? Can we resuscitate Melloware's PR and wrap up this issue?
> > Thanks,
> > Josh.
> >
> > -----Original Message-----
> > From: Gary Gregory <ga...@gmail.com>
> > Sent: Monday, September 16, 2024 4:26 PM
> > To: Commons Developers List <de...@commons.apache.org>
> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
> >
> >
> > CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
> >
> >
> >
> > Which points to:
> > https://issu/
> > es.apache.org%2Fjira%2Fbrowse%2FBEANUTILS-509&data=05%7C02%7CJosh.Fenl
> > ason%40veritas.com%7C594b7769bda54bbd892d08dcdd9ba40c%7Cfc8e13c0422c4c
> > 55b3eaca318e6cac32%7C0%7C0%7C638628907749754165%7CUnknown%7CTWFpbGZsb3
> > d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7
> > C0%7C%7C%7C&sdata=SGR%2B0SfgS%2F7gmd8VCPOlTmmlwQQNNp77J05QI5fFaBc%3D&r
> > eserved=0
> >
> > Gary
> >
> > On Mon, Sep 16, 2024, 5:24 PM Gary Gregory <ga...@gmail.com> wrote:
> >
> > > Related:
> > > https://issu/
> > > es.apache.org%2Fjira%2Fbrowse%2FCOLLECTIONS-700&data=05%7C02%7CJosh.
> > > Fe
> > > nlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c042
> > > 2c
> > > 4c55b3eaca318e6cac32%7C0%7C0%7C638621188049710140%7CUnknown%7CTWFpbG
> > > Zs
> > > b3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%
> > > 3D
> > > %7C0%7C%7C%7C&sdata=UspxFjiRSi4rTJSW7wNHs2XESjZ0Cvbm350jrmVMD3I%3D&r
> > > es
> > > erved=0
> > >
> > > Gary
> > >
> > > On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
> > >
> > >> IMO I think it ok...so Gary which part do you feel unsafe in it ？I
> > >> must admit I am not that familiar to lawyer things..
> > >> ________________________________
> > >> From: Melloware Inc <me...@gmail.com>
> > >> Sent: Saturday, September 14, 2024 9:18:08 PM
> > >> To: Commons Developers List <de...@commons.apache.org>
> > >> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> ConcurrentHashMap
> > >>
> > >> Well in my PR the license in the file says this.
> > >>
> > >> Written by Doug Lea with assistance from members of JCP JSR-166
> > >> Expert Group and released to the public domain, as explained at
> > >> @see <a
> > >> href="https://nam12.safelinks.protection.outlook.com/?url=http%3A%252
> > >> F%25
> > >> 2Fcreativecommons.org%2Flicenses%2Fpublicdomain&data=05%7C02%7CJosh
> > >> .F
> > >> enlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c0
> > >> 42
> > >> 2c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049723177%7CUnknown%7CTWF
> > >> pb
> > >> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6
> > >> Mn
> > >> 0%3D%7C0%7C%7C%7C&sdata=Vgo5BT3zXWMHgn4iv1PKrw0kZd2JMUo2V1o78%2FMuF
> > >> po
> > >> %3D&reserved=0">Creative
> > >> Commons</a>
> > >>
> > >> it says its public domain.
> > >>
> > >> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
> > >>
> > >> > why not write an email to lea ..maybe he be so kind that would be
> > >> > glad
> > >> to
> > >> > offer one mit-like license copy of that class
> > >> > ________________________________
> > >> > From: Melloware Inc <me...@gmail.com>
> > >> > Sent: Saturday, September 14, 2024 8:32:38 PM
> > >> > To: Commons Developers List <de...@commons.apache.org>
> > >> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> > ConcurrentHashMap
> > >> >
> > >> > I submitted this PR years ago using ConcurrentWeakKeyHashMap from
> > >> > Doug
> > >> Lea
> > >> > and the Netty team but Gary had concerns from legal about being
> > >> > able to
> > >> use
> > >> > it.
> > >> >
> > >> > PR is here:
> > >> > https://g/
> > >> > ithub.com%2Fapache%2Fcommons-beanutils%2Fpull%2F56&data=05%7C02%7
> > >> > CJ
> > >> > osh.Fenlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cf
> > >> > c8
> > >> > e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049734927%7CUnkn
> > >> > ow
> > >> > n%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1ha
> > >> > Ww
> > >> > iLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=ERcuJKHJW%2F7plLn9Pj5nUaA4rFCu
> > >> > kB
> > >> > s8V6WP%2BABCzRc%3D&reserved=0
> > >> >
> > >> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> > >> >
> > >> > > I thought old d-lea bro have a concurrent weak hashmap
> > >> > > implement somewhere...why not just ask him after all nobody
> > >> > > here more expert than him in this area
> > >> > > ________________________________
> > >> > > From: Niall Pemberton <ni...@gmail.com>
> > >> > > Sent: Saturday, September 14, 2024 1:46:32 PM
> > >> > > To: Commons Developers List <de...@commons.apache.org>
> > >> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> > >> > > ConcurrentHashMap
> > >> > >
> > >> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory
> > >> > > <gg...@apache.org>
> > >> > wrote:
> > >> > >
> > >> > > > Hi All,
> > >> > > >
> > >> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> > >> > WeakFastHashMap
> > >> > > > with the stock ConcurrentHashMap.
> > >> > > >
> > >> > > > If you think this is a bad idea, please tell us why.
> > >> > >
> > >> > >
> > >> > > It’s a good idea for the “fast” part, but the “weak” aspect
> > >> > > also
> > >> needs to
> > >> > > be retained - so in principle yes, but the implementation
> > >> > > detail
> > >> matters.
> > >> > >
> > >> > > BeanUtils caused memory issues in multiple ClassLoader
> > >> > > environments
> > >> (e.g.
> > >> > > WebApp containers) because of strong references to classes
> > >> > > prevented
> > >> them
> > >> > > from being garbage collected if the app was reloaded/restarted.
> > >> > > So
> > >> this
> > >> > > weak map was introduced to resolve that issue.
> > >> > >
> > >> > > Niall
> > >> > >
> > >> > >
> > >> > > >
> > >> > > > Gary
> > >> > > >
> > >> > > >
> > >> -------------------------------------------------------------------
> > >> --
> > >> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > >> > > > For additional commands, e-mail: dev-help@commons.apache.org
> > >> > > >
> > >> > > >
> > >> > >
> > >> >
> > >> >
> > >> > --
> > >> > ==============================
> > >> > Melloware
> > >> > mellowaredev@gmail.com
> > >> > http://me/
> > >> > lloware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9
> > >> > a6
> > >> > a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%
> > >> > 7C
> > >> > 0%7C638621188049744703%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMD
> > >> > Ai
> > >> > LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata
> > >> > =o
> > >> > 6%2BhNto8to7f4E1GZtN%2BpFVpoFx0b4gI4Exy90G7apk%3D&reserved=0
> > >> > ==============================
> > >> >
> > >>
> > >>
> > >> --
> > >> ==============================
> > >> Melloware
> > >> mellowaredev@gmail.com
> > >> http://mell/
> > >> oware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6a1
> > >> 3a
> > >> 540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C
> > >> 63
> > >> 8621188049751739%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIj
> > >> oi
> > >> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=Iof1UJppc
> > >> Gh
> > >> PV%2F5T2Lmo5i%2BX1nA%2F0V1M7eyEqjvHtUQ%3D&reserved=0
> > >> ==============================
> > >>
> > >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
RE: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Josh Fenlason <Jo...@veritas.com.INVALID>.
Excellent, thanks for that encouraging update! Is there anything I can assist with?
Josh.
-----Original Message-----
From: Gary Gregory <ga...@gmail.com>
Sent: Wednesday, September 25, 2024 2:52 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
FWIW, I have a patch in progress using the Apache Grooy's version of the class, so licensing will not be an issue, Apache to Apache.
If you look at the recent commits in git master, you'll see that I've started adding missing tests we need before we add anything else...
I also have tests written that fit in the existing test framework.
These will likely need to be beefed up. I saw zero tests for this class in the Groovy code base.
Stay tuned.
Gary
On Wed, Sep 25, 2024 at 3:37 PM Josh Fenlason <Jo...@veritas.com.invalid> wrote:
>
> Are legal concerns of using Netty's ConcurrentWeakKeyHashMap the only issue with Melloware's PR (https://github.com/apache/commons-beanutils/pull/56) from a couple of years ago? That PR mentions http://creativecommons.org/licenses/publicdomain for details on the license, which unfortunately no longer works. However, looking at the Netty source (https://github.com/netty/netty/blob/3.9/src/main/java/org/jboss/netty/util/internal/ConcurrentWeakKeyHashMap.java), they have an Apache 2 license. Doesn't that alleviate any license concerns? Can we resuscitate Melloware's PR and wrap up this issue?
> Thanks,
> Josh.
>
> -----Original Message-----
> From: Gary Gregory <ga...@gmail.com>
> Sent: Monday, September 16, 2024 4:26 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
>
> CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
>
>
>
> Which points to:
> https://issu/
> es.apache.org%2Fjira%2Fbrowse%2FBEANUTILS-509&data=05%7C02%7CJosh.Fenl
> ason%40veritas.com%7C594b7769bda54bbd892d08dcdd9ba40c%7Cfc8e13c0422c4c
> 55b3eaca318e6cac32%7C0%7C0%7C638628907749754165%7CUnknown%7CTWFpbGZsb3
> d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7
> C0%7C%7C%7C&sdata=SGR%2B0SfgS%2F7gmd8VCPOlTmmlwQQNNp77J05QI5fFaBc%3D&r
> eserved=0
>
> Gary
>
> On Mon, Sep 16, 2024, 5:24 PM Gary Gregory <ga...@gmail.com> wrote:
>
> > Related:
> > https://issu/
> > es.apache.org%2Fjira%2Fbrowse%2FCOLLECTIONS-700&data=05%7C02%7CJosh.
> > Fe
> > nlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c042
> > 2c
> > 4c55b3eaca318e6cac32%7C0%7C0%7C638621188049710140%7CUnknown%7CTWFpbG
> > Zs
> > b3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%
> > 3D
> > %7C0%7C%7C%7C&sdata=UspxFjiRSi4rTJSW7wNHs2XESjZ0Cvbm350jrmVMD3I%3D&r
> > es
> > erved=0
> >
> > Gary
> >
> > On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
> >
> >> IMO I think it ok...so Gary which part do you feel unsafe in it ？I
> >> must admit I am not that familiar to lawyer things..
> >> ________________________________
> >> From: Melloware Inc <me...@gmail.com>
> >> Sent: Saturday, September 14, 2024 9:18:08 PM
> >> To: Commons Developers List <de...@commons.apache.org>
> >> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> >> ConcurrentHashMap
> >>
> >> Well in my PR the license in the file says this.
> >>
> >> Written by Doug Lea with assistance from members of JCP JSR-166
> >> Expert Group and released to the public domain, as explained at
> >> @see <a
> >> href="https://nam12.safelinks.protection.outlook.com/?url=http%3A%252
> >> F%25
> >> 2Fcreativecommons.org%2Flicenses%2Fpublicdomain&data=05%7C02%7CJosh
> >> .F
> >> enlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c0
> >> 42
> >> 2c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049723177%7CUnknown%7CTWF
> >> pb
> >> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6
> >> Mn
> >> 0%3D%7C0%7C%7C%7C&sdata=Vgo5BT3zXWMHgn4iv1PKrw0kZd2JMUo2V1o78%2FMuF
> >> po
> >> %3D&reserved=0">Creative
> >> Commons</a>
> >>
> >> it says its public domain.
> >>
> >> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
> >>
> >> > why not write an email to lea ..maybe he be so kind that would be
> >> > glad
> >> to
> >> > offer one mit-like license copy of that class
> >> > ________________________________
> >> > From: Melloware Inc <me...@gmail.com>
> >> > Sent: Saturday, September 14, 2024 8:32:38 PM
> >> > To: Commons Developers List <de...@commons.apache.org>
> >> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> >> > ConcurrentHashMap
> >> >
> >> > I submitted this PR years ago using ConcurrentWeakKeyHashMap from
> >> > Doug
> >> Lea
> >> > and the Netty team but Gary had concerns from legal about being
> >> > able to
> >> use
> >> > it.
> >> >
> >> > PR is here:
> >> > https://g/
> >> > ithub.com%2Fapache%2Fcommons-beanutils%2Fpull%2F56&data=05%7C02%7
> >> > CJ
> >> > osh.Fenlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cf
> >> > c8
> >> > e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049734927%7CUnkn
> >> > ow
> >> > n%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1ha
> >> > Ww
> >> > iLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=ERcuJKHJW%2F7plLn9Pj5nUaA4rFCu
> >> > kB
> >> > s8V6WP%2BABCzRc%3D&reserved=0
> >> >
> >> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> >> >
> >> > > I thought old d-lea bro have a concurrent weak hashmap
> >> > > implement somewhere...why not just ask him after all nobody
> >> > > here more expert than him in this area
> >> > > ________________________________
> >> > > From: Niall Pemberton <ni...@gmail.com>
> >> > > Sent: Saturday, September 14, 2024 1:46:32 PM
> >> > > To: Commons Developers List <de...@commons.apache.org>
> >> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> >> > > ConcurrentHashMap
> >> > >
> >> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory
> >> > > <gg...@apache.org>
> >> > wrote:
> >> > >
> >> > > > Hi All,
> >> > > >
> >> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> >> > WeakFastHashMap
> >> > > > with the stock ConcurrentHashMap.
> >> > > >
> >> > > > If you think this is a bad idea, please tell us why.
> >> > >
> >> > >
> >> > > It’s a good idea for the “fast” part, but the “weak” aspect
> >> > > also
> >> needs to
> >> > > be retained - so in principle yes, but the implementation
> >> > > detail
> >> matters.
> >> > >
> >> > > BeanUtils caused memory issues in multiple ClassLoader
> >> > > environments
> >> (e.g.
> >> > > WebApp containers) because of strong references to classes
> >> > > prevented
> >> them
> >> > > from being garbage collected if the app was reloaded/restarted.
> >> > > So
> >> this
> >> > > weak map was introduced to resolve that issue.
> >> > >
> >> > > Niall
> >> > >
> >> > >
> >> > > >
> >> > > > Gary
> >> > > >
> >> > > >
> >> -------------------------------------------------------------------
> >> --
> >> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> >> > > > For additional commands, e-mail: dev-help@commons.apache.org
> >> > > >
> >> > > >
> >> > >
> >> >
> >> >
> >> > --
> >> > ==============================
> >> > Melloware
> >> > mellowaredev@gmail.com
> >> > http://me/
> >> > lloware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9
> >> > a6
> >> > a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%
> >> > 7C
> >> > 0%7C638621188049744703%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMD
> >> > Ai
> >> > LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata
> >> > =o
> >> > 6%2BhNto8to7f4E1GZtN%2BpFVpoFx0b4gI4Exy90G7apk%3D&reserved=0
> >> > ==============================
> >> >
> >>
> >>
> >> --
> >> ==============================
> >> Melloware
> >> mellowaredev@gmail.com
> >> http://mell/
> >> oware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6a1
> >> 3a
> >> 540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C
> >> 63
> >> 8621188049751739%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIj
> >> oi
> >> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=Iof1UJppc
> >> Gh
> >> PV%2F5T2Lmo5i%2BX1nA%2F0V1M7eyEqjvHtUQ%3D&reserved=0
> >> ==============================
> >>
> >
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Gary Gregory <ga...@gmail.com>.
FWIW, I have a patch in progress using the Apache Grooy's version of
the class, so licensing will not be an issue, Apache to Apache.
If you look at the recent commits in git master, you'll see that I've
started adding missing tests we need before we add anything else...
I also have tests written that fit in the existing test framework.
These will likely need to be beefed up. I saw zero tests for this
class in the Groovy code base.
Stay tuned.
Gary
On Wed, Sep 25, 2024 at 3:37 PM Josh Fenlason
<Jo...@veritas.com.invalid> wrote:
>
> Are legal concerns of using Netty's ConcurrentWeakKeyHashMap the only issue with Melloware's PR (https://github.com/apache/commons-beanutils/pull/56) from a couple of years ago? That PR mentions http://creativecommons.org/licenses/publicdomain for details on the license, which unfortunately no longer works. However, looking at the Netty source (https://github.com/netty/netty/blob/3.9/src/main/java/org/jboss/netty/util/internal/ConcurrentWeakKeyHashMap.java), they have an Apache 2 license. Doesn't that alleviate any license concerns? Can we resuscitate Melloware's PR and wrap up this issue?
> Thanks,
> Josh.
>
> -----Original Message-----
> From: Gary Gregory <ga...@gmail.com>
> Sent: Monday, September 16, 2024 4:26 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
>
> CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
>
>
>
> Which points to: https://issues.apache.org/jira/browse/BEANUTILS-509
>
> Gary
>
> On Mon, Sep 16, 2024, 5:24 PM Gary Gregory <ga...@gmail.com> wrote:
>
> > Related:
> > https://issu/
> > es.apache.org%2Fjira%2Fbrowse%2FCOLLECTIONS-700&data=05%7C02%7CJosh.Fe
> > nlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c
> > 4c55b3eaca318e6cac32%7C0%7C0%7C638621188049710140%7CUnknown%7CTWFpbGZs
> > b3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D
> > %7C0%7C%7C%7C&sdata=UspxFjiRSi4rTJSW7wNHs2XESjZ0Cvbm350jrmVMD3I%3D&res
> > erved=0
> >
> > Gary
> >
> > On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
> >
> >> IMO I think it ok...so Gary which part do you feel unsafe in it ？I
> >> must admit I am not that familiar to lawyer things..
> >> ________________________________
> >> From: Melloware Inc <me...@gmail.com>
> >> Sent: Saturday, September 14, 2024 9:18:08 PM
> >> To: Commons Developers List <de...@commons.apache.org>
> >> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> >> ConcurrentHashMap
> >>
> >> Well in my PR the license in the file says this.
> >>
> >> Written by Doug Lea with assistance from members of JCP JSR-166
> >> Expert Group and released to the public domain, as explained at @see
> >> <a
> >> href="https://nam12.safelinks.protection.outlook.com/?url=http%3A%2F%25
> >> 2Fcreativecommons.org%2Flicenses%2Fpublicdomain&data=05%7C02%7CJosh.F
> >> enlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c042
> >> 2c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049723177%7CUnknown%7CTWFpb
> >> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn
> >> 0%3D%7C0%7C%7C%7C&sdata=Vgo5BT3zXWMHgn4iv1PKrw0kZd2JMUo2V1o78%2FMuFpo
> >> %3D&reserved=0">Creative
> >> Commons</a>
> >>
> >> it says its public domain.
> >>
> >> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
> >>
> >> > why not write an email to lea ..maybe he be so kind that would be
> >> > glad
> >> to
> >> > offer one mit-like license copy of that class
> >> > ________________________________
> >> > From: Melloware Inc <me...@gmail.com>
> >> > Sent: Saturday, September 14, 2024 8:32:38 PM
> >> > To: Commons Developers List <de...@commons.apache.org>
> >> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> >> > ConcurrentHashMap
> >> >
> >> > I submitted this PR years ago using ConcurrentWeakKeyHashMap from
> >> > Doug
> >> Lea
> >> > and the Netty team but Gary had concerns from legal about being
> >> > able to
> >> use
> >> > it.
> >> >
> >> > PR is here:
> >> > https://g/
> >> > ithub.com%2Fapache%2Fcommons-beanutils%2Fpull%2F56&data=05%7C02%7CJ
> >> > osh.Fenlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8
> >> > e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049734927%7CUnknow
> >> > n%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWw
> >> > iLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=ERcuJKHJW%2F7plLn9Pj5nUaA4rFCukB
> >> > s8V6WP%2BABCzRc%3D&reserved=0
> >> >
> >> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> >> >
> >> > > I thought old d-lea bro have a concurrent weak hashmap implement
> >> > > somewhere...why not just ask him after all nobody here more
> >> > > expert than him in this area ________________________________
> >> > > From: Niall Pemberton <ni...@gmail.com>
> >> > > Sent: Saturday, September 14, 2024 1:46:32 PM
> >> > > To: Commons Developers List <de...@commons.apache.org>
> >> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
> >> > > ConcurrentHashMap
> >> > >
> >> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory
> >> > > <gg...@apache.org>
> >> > wrote:
> >> > >
> >> > > > Hi All,
> >> > > >
> >> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> >> > WeakFastHashMap
> >> > > > with the stock ConcurrentHashMap.
> >> > > >
> >> > > > If you think this is a bad idea, please tell us why.
> >> > >
> >> > >
> >> > > It’s a good idea for the “fast” part, but the “weak” aspect also
> >> needs to
> >> > > be retained - so in principle yes, but the implementation detail
> >> matters.
> >> > >
> >> > > BeanUtils caused memory issues in multiple ClassLoader
> >> > > environments
> >> (e.g.
> >> > > WebApp containers) because of strong references to classes
> >> > > prevented
> >> them
> >> > > from being garbage collected if the app was reloaded/restarted.
> >> > > So
> >> this
> >> > > weak map was introduced to resolve that issue.
> >> > >
> >> > > Niall
> >> > >
> >> > >
> >> > > >
> >> > > > Gary
> >> > > >
> >> > > >
> >> ---------------------------------------------------------------------
> >> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> >> > > > For additional commands, e-mail: dev-help@commons.apache.org
> >> > > >
> >> > > >
> >> > >
> >> >
> >> >
> >> > --
> >> > ==============================
> >> > Melloware
> >> > mellowaredev@gmail.com
> >> > http://me/
> >> > lloware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6
> >> > a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C
> >> > 0%7C638621188049744703%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAi
> >> > LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=o
> >> > 6%2BhNto8to7f4E1GZtN%2BpFVpoFx0b4gI4Exy90G7apk%3D&reserved=0
> >> > ==============================
> >> >
> >>
> >>
> >> --
> >> ==============================
> >> Melloware
> >> mellowaredev@gmail.com
> >> http://mell/
> >> oware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6a13a
> >> 540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C63
> >> 8621188049751739%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
> >> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=Iof1UJppcGh
> >> PV%2F5T2Lmo5i%2BX1nA%2F0V1M7eyEqjvHtUQ%3D&reserved=0
> >> ==============================
> >>
> >
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
RE: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Josh Fenlason <Jo...@veritas.com.INVALID>.
Are legal concerns of using Netty's ConcurrentWeakKeyHashMap the only issue with Melloware's PR (https://github.com/apache/commons-beanutils/pull/56) from a couple of years ago? That PR mentions http://creativecommons.org/licenses/publicdomain for details on the license, which unfortunately no longer works. However, looking at the Netty source (https://github.com/netty/netty/blob/3.9/src/main/java/org/jboss/netty/util/internal/ConcurrentWeakKeyHashMap.java), they have an Apache 2 license. Doesn't that alleviate any license concerns? Can we resuscitate Melloware's PR and wrap up this issue?
Thanks,
Josh.
-----Original Message-----
From: Gary Gregory <ga...@gmail.com>
Sent: Monday, September 16, 2024 4:26 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
Which points to: https://issues.apache.org/jira/browse/BEANUTILS-509
Gary
On Mon, Sep 16, 2024, 5:24 PM Gary Gregory <ga...@gmail.com> wrote:
> Related:
> https://issu/
> es.apache.org%2Fjira%2Fbrowse%2FCOLLECTIONS-700&data=05%7C02%7CJosh.Fe
> nlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c
> 4c55b3eaca318e6cac32%7C0%7C0%7C638621188049710140%7CUnknown%7CTWFpbGZs
> b3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D
> %7C0%7C%7C%7C&sdata=UspxFjiRSi4rTJSW7wNHs2XESjZ0Cvbm350jrmVMD3I%3D&res
> erved=0
>
> Gary
>
> On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
>
>> IMO I think it ok...so Gary which part do you feel unsafe in it ？I
>> must admit I am not that familiar to lawyer things..
>> ________________________________
>> From: Melloware Inc <me...@gmail.com>
>> Sent: Saturday, September 14, 2024 9:18:08 PM
>> To: Commons Developers List <de...@commons.apache.org>
>> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
>> ConcurrentHashMap
>>
>> Well in my PR the license in the file says this.
>>
>> Written by Doug Lea with assistance from members of JCP JSR-166
>> Expert Group and released to the public domain, as explained at @see
>> <a
>> href="https://nam12.safelinks.protection.outlook.com/?url=http%3A%2F%25
>> 2Fcreativecommons.org%2Flicenses%2Fpublicdomain&data=05%7C02%7CJosh.F
>> enlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8e13c042
>> 2c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049723177%7CUnknown%7CTWFpb
>> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn
>> 0%3D%7C0%7C%7C%7C&sdata=Vgo5BT3zXWMHgn4iv1PKrw0kZd2JMUo2V1o78%2FMuFpo
>> %3D&reserved=0">Creative
>> Commons</a>
>>
>> it says its public domain.
>>
>> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
>>
>> > why not write an email to lea ..maybe he be so kind that would be
>> > glad
>> to
>> > offer one mit-like license copy of that class
>> > ________________________________
>> > From: Melloware Inc <me...@gmail.com>
>> > Sent: Saturday, September 14, 2024 8:32:38 PM
>> > To: Commons Developers List <de...@commons.apache.org>
>> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
>> > ConcurrentHashMap
>> >
>> > I submitted this PR years ago using ConcurrentWeakKeyHashMap from
>> > Doug
>> Lea
>> > and the Netty team but Gary had concerns from legal about being
>> > able to
>> use
>> > it.
>> >
>> > PR is here:
>> > https://g/
>> > ithub.com%2Fapache%2Fcommons-beanutils%2Fpull%2F56&data=05%7C02%7CJ
>> > osh.Fenlason%40veritas.com%7Ca17c9a6a13a540debe7f08dcd6963dfb%7Cfc8
>> > e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638621188049734927%7CUnknow
>> > n%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWw
>> > iLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=ERcuJKHJW%2F7plLn9Pj5nUaA4rFCukB
>> > s8V6WP%2BABCzRc%3D&reserved=0
>> >
>> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
>> >
>> > > I thought old d-lea bro have a concurrent weak hashmap implement
>> > > somewhere...why not just ask him after all nobody here more
>> > > expert than him in this area ________________________________
>> > > From: Niall Pemberton <ni...@gmail.com>
>> > > Sent: Saturday, September 14, 2024 1:46:32 PM
>> > > To: Commons Developers List <de...@commons.apache.org>
>> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs
>> > > ConcurrentHashMap
>> > >
>> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory
>> > > <gg...@apache.org>
>> > wrote:
>> > >
>> > > > Hi All,
>> > > >
>> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
>> > WeakFastHashMap
>> > > > with the stock ConcurrentHashMap.
>> > > >
>> > > > If you think this is a bad idea, please tell us why.
>> > >
>> > >
>> > > It’s a good idea for the “fast” part, but the “weak” aspect also
>> needs to
>> > > be retained - so in principle yes, but the implementation detail
>> matters.
>> > >
>> > > BeanUtils caused memory issues in multiple ClassLoader
>> > > environments
>> (e.g.
>> > > WebApp containers) because of strong references to classes
>> > > prevented
>> them
>> > > from being garbage collected if the app was reloaded/restarted.
>> > > So
>> this
>> > > weak map was introduced to resolve that issue.
>> > >
>> > > Niall
>> > >
>> > >
>> > > >
>> > > > Gary
>> > > >
>> > > >
>> ---------------------------------------------------------------------
>> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> > > > For additional commands, e-mail: dev-help@commons.apache.org
>> > > >
>> > > >
>> > >
>> >
>> >
>> > --
>> > ==============================
>> > Melloware
>> > mellowaredev@gmail.com
>> > http://me/
>> > lloware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6
>> > a13a540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C
>> > 0%7C638621188049744703%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAi
>> > LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=o
>> > 6%2BhNto8to7f4E1GZtN%2BpFVpoFx0b4gI4Exy90G7apk%3D&reserved=0
>> > ==============================
>> >
>>
>>
>> --
>> ==============================
>> Melloware
>> mellowaredev@gmail.com
>> http://mell/
>> oware.com%2F&data=05%7C02%7CJosh.Fenlason%40veritas.com%7Ca17c9a6a13a
>> 540debe7f08dcd6963dfb%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C63
>> 8621188049751739%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
>> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=Iof1UJppcGh
>> PV%2F5T2Lmo5i%2BX1nA%2F0V1M7eyEqjvHtUQ%3D&reserved=0
>> ==============================
>>
>
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Gary Gregory <ga...@gmail.com>.
Which points to: https://issues.apache.org/jira/browse/BEANUTILS-509
Gary
On Mon, Sep 16, 2024, 5:24 PM Gary Gregory <ga...@gmail.com> wrote:
> Related: https://issues.apache.org/jira/browse/COLLECTIONS-700
>
> Gary
>
> On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
>
>> IMO I think it ok...so Gary which part do you feel unsafe in it ？I must
>> admit I am not that familiar to lawyer things..
>> ________________________________
>> From: Melloware Inc <me...@gmail.com>
>> Sent: Saturday, September 14, 2024 9:18:08 PM
>> To: Commons Developers List <de...@commons.apache.org>
>> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>>
>> Well in my PR the license in the file says this.
>>
>> Written by Doug Lea with assistance from members of JCP JSR-166
>> Expert Group and released to the public domain, as explained at
>> @see <a href="http://creativecommons.org/licenses/publicdomain">Creative
>> Commons</a>
>>
>> it says its public domain.
>>
>> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
>>
>> > why not write an email to lea ..maybe he be so kind that would be glad
>> to
>> > offer one mit-like license copy of that class
>> > ________________________________
>> > From: Melloware Inc <me...@gmail.com>
>> > Sent: Saturday, September 14, 2024 8:32:38 PM
>> > To: Commons Developers List <de...@commons.apache.org>
>> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>> >
>> > I submitted this PR years ago using ConcurrentWeakKeyHashMap from Doug
>> Lea
>> > and the Netty team but Gary had concerns from legal about being able to
>> use
>> > it.
>> >
>> > PR is here: https://github.com/apache/commons-beanutils/pull/56
>> >
>> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
>> >
>> > > I thought old d-lea bro have a concurrent weak hashmap implement
>> > > somewhere...why not just ask him
>> > > after all nobody here more expert than him in this area
>> > > ________________________________
>> > > From: Niall Pemberton <ni...@gmail.com>
>> > > Sent: Saturday, September 14, 2024 1:46:32 PM
>> > > To: Commons Developers List <de...@commons.apache.org>
>> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>> > >
>> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org>
>> > wrote:
>> > >
>> > > > Hi All,
>> > > >
>> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
>> > WeakFastHashMap
>> > > > with the stock ConcurrentHashMap.
>> > > >
>> > > > If you think this is a bad idea, please tell us why.
>> > >
>> > >
>> > > It’s a good idea for the “fast” part, but the “weak” aspect also
>> needs to
>> > > be retained - so in principle yes, but the implementation detail
>> matters.
>> > >
>> > > BeanUtils caused memory issues in multiple ClassLoader environments
>> (e.g.
>> > > WebApp containers) because of strong references to classes prevented
>> them
>> > > from being garbage collected if the app was reloaded/restarted. So
>> this
>> > > weak map was introduced to resolve that issue.
>> > >
>> > > Niall
>> > >
>> > >
>> > > >
>> > > > Gary
>> > > >
>> > > >
>> ---------------------------------------------------------------------
>> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> > > > For additional commands, e-mail: dev-help@commons.apache.org
>> > > >
>> > > >
>> > >
>> >
>> >
>> > --
>> > ==============================
>> > Melloware
>> > mellowaredev@gmail.com
>> > http://melloware.com
>> > ==============================
>> >
>>
>>
>> --
>> ==============================
>> Melloware
>> mellowaredev@gmail.com
>> http://melloware.com
>> ==============================
>>
>
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Gary Gregory <ga...@gmail.com>.
Related: https://issues.apache.org/jira/browse/COLLECTIONS-700
Gary
On Sat, Sep 14, 2024, 10:16 AM Xeno Amess <xe...@gmail.com> wrote:
> IMO I think it ok...so Gary which part do you feel unsafe in it ？I must
> admit I am not that familiar to lawyer things..
> ________________________________
> From: Melloware Inc <me...@gmail.com>
> Sent: Saturday, September 14, 2024 9:18:08 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
> Well in my PR the license in the file says this.
>
> Written by Doug Lea with assistance from members of JCP JSR-166
> Expert Group and released to the public domain, as explained at
> @see <a href="http://creativecommons.org/licenses/publicdomain">Creative
> Commons</a>
>
> it says its public domain.
>
> On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
>
> > why not write an email to lea ..maybe he be so kind that would be glad to
> > offer one mit-like license copy of that class
> > ________________________________
> > From: Melloware Inc <me...@gmail.com>
> > Sent: Saturday, September 14, 2024 8:32:38 PM
> > To: Commons Developers List <de...@commons.apache.org>
> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
> >
> > I submitted this PR years ago using ConcurrentWeakKeyHashMap from Doug
> Lea
> > and the Netty team but Gary had concerns from legal about being able to
> use
> > it.
> >
> > PR is here: https://github.com/apache/commons-beanutils/pull/56
> >
> > On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> >
> > > I thought old d-lea bro have a concurrent weak hashmap implement
> > > somewhere...why not just ask him
> > > after all nobody here more expert than him in this area
> > > ________________________________
> > > From: Niall Pemberton <ni...@gmail.com>
> > > Sent: Saturday, September 14, 2024 1:46:32 PM
> > > To: Commons Developers List <de...@commons.apache.org>
> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
> > >
> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org>
> > wrote:
> > >
> > > > Hi All,
> > > >
> > > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> > WeakFastHashMap
> > > > with the stock ConcurrentHashMap.
> > > >
> > > > If you think this is a bad idea, please tell us why.
> > >
> > >
> > > It’s a good idea for the “fast” part, but the “weak” aspect also needs
> to
> > > be retained - so in principle yes, but the implementation detail
> matters.
> > >
> > > BeanUtils caused memory issues in multiple ClassLoader environments
> (e.g.
> > > WebApp containers) because of strong references to classes prevented
> them
> > > from being garbage collected if the app was reloaded/restarted. So this
> > > weak map was introduced to resolve that issue.
> > >
> > > Niall
> > >
> > >
> > > >
> > > > Gary
> > > >
> > > > ---------------------------------------------------------------------
> > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > > > For additional commands, e-mail: dev-help@commons.apache.org
> > > >
> > > >
> > >
> >
> >
> > --
> > ==============================
> > Melloware
> > mellowaredev@gmail.com
> > http://melloware.com
> > ==============================
> >
>
>
> --
> ==============================
> Melloware
> mellowaredev@gmail.com
> http://melloware.com
> ==============================
>
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Xeno Amess <xe...@gmail.com>.
IMO I think it ok...so Gary which part do you feel unsafe in it ？I must admit I am not that familiar to lawyer things..
________________________________
From: Melloware Inc <me...@gmail.com>
Sent: Saturday, September 14, 2024 9:18:08 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Well in my PR the license in the file says this.
Written by Doug Lea with assistance from members of JCP JSR-166
Expert Group and released to the public domain, as explained at
@see <a href="http://creativecommons.org/licenses/publicdomain">Creative
Commons</a>
it says its public domain.
On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
> why not write an email to lea ..maybe he be so kind that would be glad to
> offer one mit-like license copy of that class
> ________________________________
> From: Melloware Inc <me...@gmail.com>
> Sent: Saturday, September 14, 2024 8:32:38 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
> I submitted this PR years ago using ConcurrentWeakKeyHashMap from Doug Lea
> and the Netty team but Gary had concerns from legal about being able to use
> it.
>
> PR is here: https://github.com/apache/commons-beanutils/pull/56
>
> On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
>
> > I thought old d-lea bro have a concurrent weak hashmap implement
> > somewhere...why not just ask him
> > after all nobody here more expert than him in this area
> > ________________________________
> > From: Niall Pemberton <ni...@gmail.com>
> > Sent: Saturday, September 14, 2024 1:46:32 PM
> > To: Commons Developers List <de...@commons.apache.org>
> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
> >
> > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org>
> wrote:
> >
> > > Hi All,
> > >
> > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> WeakFastHashMap
> > > with the stock ConcurrentHashMap.
> > >
> > > If you think this is a bad idea, please tell us why.
> >
> >
> > It’s a good idea for the “fast” part, but the “weak” aspect also needs to
> > be retained - so in principle yes, but the implementation detail matters.
> >
> > BeanUtils caused memory issues in multiple ClassLoader environments (e.g.
> > WebApp containers) because of strong references to classes prevented them
> > from being garbage collected if the app was reloaded/restarted. So this
> > weak map was introduced to resolve that issue.
> >
> > Niall
> >
> >
> > >
> > > Gary
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > > For additional commands, e-mail: dev-help@commons.apache.org
> > >
> > >
> >
>
>
> --
> ==============================
> Melloware
> mellowaredev@gmail.com
> http://melloware.com
> ==============================
>
--
==============================
Melloware
mellowaredev@gmail.com
http://melloware.com
==============================
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Melloware Inc <me...@gmail.com>.
Well in my PR the license in the file says this.
Written by Doug Lea with assistance from members of JCP JSR-166
Expert Group and released to the public domain, as explained at
@see <a href="http://creativecommons.org/licenses/publicdomain">Creative
Commons</a>
it says its public domain.
On Sat, Sep 14, 2024 at 9:01 AM Xeno Amess <xe...@gmail.com> wrote:
> why not write an email to lea ..maybe he be so kind that would be glad to
> offer one mit-like license copy of that class
> ________________________________
> From: Melloware Inc <me...@gmail.com>
> Sent: Saturday, September 14, 2024 8:32:38 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
> I submitted this PR years ago using ConcurrentWeakKeyHashMap from Doug Lea
> and the Netty team but Gary had concerns from legal about being able to use
> it.
>
> PR is here: https://github.com/apache/commons-beanutils/pull/56
>
> On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
>
> > I thought old d-lea bro have a concurrent weak hashmap implement
> > somewhere...why not just ask him
> > after all nobody here more expert than him in this area
> > ________________________________
> > From: Niall Pemberton <ni...@gmail.com>
> > Sent: Saturday, September 14, 2024 1:46:32 PM
> > To: Commons Developers List <de...@commons.apache.org>
> > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
> >
> > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org>
> wrote:
> >
> > > Hi All,
> > >
> > > For the upcoming 2.0.0-M1, I plan on replacing the custom
> WeakFastHashMap
> > > with the stock ConcurrentHashMap.
> > >
> > > If you think this is a bad idea, please tell us why.
> >
> >
> > It’s a good idea for the “fast” part, but the “weak” aspect also needs to
> > be retained - so in principle yes, but the implementation detail matters.
> >
> > BeanUtils caused memory issues in multiple ClassLoader environments (e.g.
> > WebApp containers) because of strong references to classes prevented them
> > from being garbage collected if the app was reloaded/restarted. So this
> > weak map was introduced to resolve that issue.
> >
> > Niall
> >
> >
> > >
> > > Gary
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > > For additional commands, e-mail: dev-help@commons.apache.org
> > >
> > >
> >
>
>
> --
> ==============================
> Melloware
> mellowaredev@gmail.com
> http://melloware.com
> ==============================
>
--
==============================
Melloware
mellowaredev@gmail.com
http://melloware.com
==============================
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Xeno Amess <xe...@gmail.com>.
why not write an email to lea ..maybe he be so kind that would be glad to offer one mit-like license copy of that class
________________________________
From: Melloware Inc <me...@gmail.com>
Sent: Saturday, September 14, 2024 8:32:38 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
I submitted this PR years ago using ConcurrentWeakKeyHashMap from Doug Lea
and the Netty team but Gary had concerns from legal about being able to use
it.
PR is here: https://github.com/apache/commons-beanutils/pull/56
On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> I thought old d-lea bro have a concurrent weak hashmap implement
> somewhere...why not just ask him
> after all nobody here more expert than him in this area
> ________________________________
> From: Niall Pemberton <ni...@gmail.com>
> Sent: Saturday, September 14, 2024 1:46:32 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
> On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org> wrote:
>
> > Hi All,
> >
> > For the upcoming 2.0.0-M1, I plan on replacing the custom WeakFastHashMap
> > with the stock ConcurrentHashMap.
> >
> > If you think this is a bad idea, please tell us why.
>
>
> It’s a good idea for the “fast” part, but the “weak” aspect also needs to
> be retained - so in principle yes, but the implementation detail matters.
>
> BeanUtils caused memory issues in multiple ClassLoader environments (e.g.
> WebApp containers) because of strong references to classes prevented them
> from being garbage collected if the app was reloaded/restarted. So this
> weak map was introduced to resolve that issue.
>
> Niall
>
>
> >
> > Gary
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
> >
> >
>
--
==============================
Melloware
mellowaredev@gmail.com
http://melloware.com
==============================
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Melloware Inc <me...@gmail.com>.
I submitted this PR years ago using ConcurrentWeakKeyHashMap from Doug Lea
and the Netty team but Gary had concerns from legal about being able to use
it.
PR is here: https://github.com/apache/commons-beanutils/pull/56
On Sat, Sep 14, 2024 at 7:22 AM Xeno Amess <xe...@gmail.com> wrote:
> I thought old d-lea bro have a concurrent weak hashmap implement
> somewhere...why not just ask him
> after all nobody here more expert than him in this area
> ________________________________
> From: Niall Pemberton <ni...@gmail.com>
> Sent: Saturday, September 14, 2024 1:46:32 PM
> To: Commons Developers List <de...@commons.apache.org>
> Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
>
> On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org> wrote:
>
> > Hi All,
> >
> > For the upcoming 2.0.0-M1, I plan on replacing the custom WeakFastHashMap
> > with the stock ConcurrentHashMap.
> >
> > If you think this is a bad idea, please tell us why.
>
>
> It’s a good idea for the “fast” part, but the “weak” aspect also needs to
> be retained - so in principle yes, but the implementation detail matters.
>
> BeanUtils caused memory issues in multiple ClassLoader environments (e.g.
> WebApp containers) because of strong references to classes prevented them
> from being garbage collected if the app was reloaded/restarted. So this
> weak map was introduced to resolve that issue.
>
> Niall
>
>
> >
> > Gary
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
> >
> >
>
--
==============================
Melloware
mellowaredev@gmail.com
http://melloware.com
==============================
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Xeno Amess <xe...@gmail.com>.
I thought old d-lea bro have a concurrent weak hashmap implement somewhere...why not just ask him
after all nobody here more expert than him in this area
________________________________
From: Niall Pemberton <ni...@gmail.com>
Sent: Saturday, September 14, 2024 1:46:32 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org> wrote:
> Hi All,
>
> For the upcoming 2.0.0-M1, I plan on replacing the custom WeakFastHashMap
> with the stock ConcurrentHashMap.
>
> If you think this is a bad idea, please tell us why.
It’s a good idea for the “fast” part, but the “weak” aspect also needs to
be retained - so in principle yes, but the implementation detail matters.
BeanUtils caused memory issues in multiple ClassLoader environments (e.g.
WebApp containers) because of strong references to classes prevented them
from being garbage collected if the app was reloaded/restarted. So this
weak map was introduced to resolve that issue.
Niall
>
> Gary
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Xeno Amess <xe...@gmail.com>.
good and should did it 5 years ago
________________________________
From: Niall Pemberton <ni...@gmail.com>
Sent: Saturday, September 14, 2024 1:46:32 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org> wrote:
> Hi All,
>
> For the upcoming 2.0.0-M1, I plan on replacing the custom WeakFastHashMap
> with the stock ConcurrentHashMap.
>
> If you think this is a bad idea, please tell us why.
It’s a good idea for the “fast” part, but the “weak” aspect also needs to
be retained - so in principle yes, but the implementation detail matters.
BeanUtils caused memory issues in multiple ClassLoader environments (e.g.
WebApp containers) because of strong references to classes prevented them
from being garbage collected if the app was reloaded/restarted. So this
weak map was introduced to resolve that issue.
Niall
>
> Gary
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Niall Pemberton <ni...@gmail.com>.
On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory <gg...@apache.org> wrote:
> Hi All,
>
> For the upcoming 2.0.0-M1, I plan on replacing the custom WeakFastHashMap
> with the stock ConcurrentHashMap.
>
> If you think this is a bad idea, please tell us why.
It’s a good idea for the “fast” part, but the “weak” aspect also needs to
be retained - so in principle yes, but the implementation detail matters.
BeanUtils caused memory issues in multiple ClassLoader environments (e.g.
WebApp containers) because of strong references to classes prevented them
from being garbage collected if the app was reloaded/restarted. So this
weak map was introduced to resolve that issue.
Niall
>
> Gary
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
RE: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Josh Fenlason <Jo...@veritas.com.INVALID>.
I concur.
-----Original Message-----
From: Melloware Inc <me...@gmail.com>
Sent: Thursday, September 12, 2024 12:25 PM
To: Commons Developers List <de...@commons.apache.org>
Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook.
I think this is a good idea.
On Thu, Sep 12, 2024 at 1:00 PM Gary D. Gregory <gg...@apache.org> wrote:
> Hi All,
>
> For the upcoming 2.0.0-M1, I plan on replacing the custom
> WeakFastHashMap with the stock ConcurrentHashMap.
>
> If you think this is a bad idea, please tell us why.
>
> Gary
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
--
==============================
Melloware
mellowaredev@gmail.com
http://melloware.com/
==============================
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [beanutils] For 2.0, WeakFastHashMap vs ConcurrentHashMap
Posted by Melloware Inc <me...@gmail.com>.
I think this is a good idea.
On Thu, Sep 12, 2024 at 1:00 PM Gary D. Gregory <gg...@apache.org> wrote:
> Hi All,
>
> For the upcoming 2.0.0-M1, I plan on replacing the custom WeakFastHashMap
> with the stock ConcurrentHashMap.
>
> If you think this is a bad idea, please tell us why.
>
> Gary
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
--
==============================
Melloware
mellowaredev@gmail.com
http://melloware.com
==============================