Sounds good. I'll keep an eye out for it. -----Original Message----- From: Gary Gregory <garydgreg...@gmail.com> Sent: Wednesday, September 25, 2024 3:22 PM To: Commons Developers List <dev@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 <josh.fenla...@veritas.com.invalid> wrote: > > Excellent, thanks for that encouraging update! Is there anything I can > assist with? > Josh. > > -----Original Message----- > From: Gary Gregory <garydgreg...@gmail.com> > Sent: Wednesday, September 25, 2024 2:52 PM > To: Commons Developers List <dev@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 > <josh.fenla...@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 <garydgreg...@gmail.com> > > Sent: Monday, September 16, 2024 4:26 PM > > To: Commons Developers List <dev@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 <garydgreg...@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 <xenoam...@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 <melloware...@gmail.com> > > >> Sent: Saturday, September 14, 2024 9:18:08 PM > > >> To: Commons Developers List <dev@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 <xenoam...@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 <melloware...@gmail.com> > > >> > Sent: Saturday, September 14, 2024 8:32:38 PM > > >> > To: Commons Developers List <dev@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 <xenoam...@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 <niall.pember...@gmail.com> > > >> > > Sent: Saturday, September 14, 2024 1:46:32 PM > > >> > > To: Commons Developers List <dev@commons.apache.org> > > >> > > Subject: Re: [beanutils] For 2.0, WeakFastHashMap vs > > >> > > ConcurrentHashMap > > >> > > > > >> > > On Thu, 12 Sep 2024 at 19:59, Gary D. Gregory > > >> > > <ggreg...@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-unsubscr...@commons.apache.org > > >> > > > For additional commands, e-mail: > > >> > > > dev-h...@commons.apache.org > > >> > > > > > >> > > > > > >> > > > > >> > > > >> > > > >> > -- > > >> > ============================== > > >> > Melloware > > >> > melloware...@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 > > >> melloware...@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-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
