[GitHub] chetanmeh commented on issue #4104: Cache empty auth results to reduce db load

chetanmeh commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-438795814 @rabbah @markusthoemmes can you review cache changes once

[GitHub] chetanmeh commented on issue #4104: Cache empty auth results to reduce db load

chetanmeh commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-437758971 > I have now limited the auth cache hard to 10 entries. Worth to be configurable ? @mhenke1 Should be fine to leave

[GitHub] chetanmeh commented on issue #4104: Cache empty auth results to reduce db load

chetanmeh commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-437053186 @markusthoemmes All caches are currently configured with `softValues` thereby the entries are evicted (apart from expiry) on

[GitHub] chetanmeh commented on issue #4104: Cache empty auth results to reduce db load

chetanmeh commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-437044918 Caching such negative lookups may be abused to put lots of of entries in cache and put pressure on heap (causing eviction of