RFR: 1052: Memory leak in RestRequestCache

Kevin Rushforth kcr at openjdk.java.net
Wed May 26 12:57:38 UTC 2021

On Wed, 26 May 2021 12:46:39 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> I agree with Kevin, I _think_ we could probably keep at least 60 minutes of cache, if not even more. Or I might have missed something ����
> That is a good point. In my initial patch I used the scheduling frequency (which is what the watchdog uses) instead of this new configuration value, which is 10 seconds, and it worked fine on staging. I thought that was a bit excessive so I increased to a default of 5 mins, which matches the lowest "maxAge" in the cache.
> While we could certainly live with a longer interval here, 24h would still solve our problems, I don't think we have much to gain from increasing it longer than my suggested 5 mins. The nature of the bots is that they run pretty much non stop, using a lot of cpu. I can't see how running this little method will have any noticeable impact on performance, even if the cache has thousands or even millions of valid entries. It shouldn't take more than milliseconds on one of 8-32 threads (depending on bot runner config).
> If we start seeing problems, this value can always be changed in the future, or configured for any specific bot runner configuration.

Seems OK to me.


PR: https://git.openjdk.java.net/skara/pull/1167

More information about the skara-dev mailing list