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
>
>

Reply via email to