magibney commented on PR #1351: URL: https://github.com/apache/solr/pull/1351#issuecomment-1632437829
I'm proposing to go with "Option 4", which hadn't occurred to me at the time I wrote the [above comment](#issuecomment-1437385040): Move `ThinCache` impl to Solr _test_ codebase. The idea is to allow for "Option 1", but with a proof-of-concept implementation to: 1. Allow for testing the new hooks 2. Demonstrate how this could be useful 3. Use as a starting point for plugin implementations ... all without the back-compat risk of premature adoption of a concrete cache implementation. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
