nscendoni commented on PR #1128: URL: https://github.com/apache/jackrabbit-oak/pull/1128#issuecomment-1782420442
> I would take a slightly different approach and not start new threads to perform the cache update. > > See draft in https://github.com/mreutegg/jackrabbit-oak/tree/OAK-10451 > > It also introduces a maxStale configuration, though not yet added to UserConfigurationImpl, to set an upper bound for how long the cache entry can be served when it is not up to date. As mentioned, this is just a draft to illustrate the approach and unit tests are missing. @mreutegg I agree with that your approach. I added Unit tests to your classes and pushed them in this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@jackrabbit.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
