duke8253 commented on pull request #7719: URL: https://github.com/apache/trafficserver/pull/7719#issuecomment-827295645
It's not from real ats code. The implementation of test3 is in fact a proposal I made internally as well, I called it batch delete since it essentially removes multiple entries at once. It will be faster no doubt since instead of doing a remove and insert every time a new session arrives, it removes only once when the cache is full. But the problem with that is we're essentially shrinking the size of the cache. e.g. cache size is 1000, but once we reach 1000 the first 500 will be removed, so the next we search we only have 501 in the cache not the 1000 we promised. -- 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]
