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