thelabdude commented on pull request #181: URL: https://github.com/apache/solr/pull/181#issuecomment-865195272
@markrmiller I decided to remove the Cache for luke responses from this PR for now. I like your idea about pushing cache support into the LukeRequestHandler, but think that should be tackled in a separate PR. For this PR, I'm not all that convinced caching would even buy us much given requests to the `/sql` handler would be load-balanced, so unlikely to see enough cache hits to justify the added code complexity of ensuring the cached field / field type info isn't stale. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
