nickva commented on PR #5619:
URL: https://github.com/apache/couchdb/pull/5619#issuecomment-3168359990
> I think it's a mistake for caching to occur both in front of and behind
the PSE abstraction. caching the Props (and using that cache) in the #db{}
record where we already cache the SecObj is the better way imo.
This is meant a cache layer just above couch_file. The lower layer
(bt_engine) knows better how to cache these things and keep them in sync. Based
on the internals of the engine it can be easier to do (check file pointers,
etc). This is also the place potentially put a btree roots cache (I might try
that next, it might not be too bad to add).
In general I'd remove the security caching in the db record, except that we
use that for get_db fabric and we'd have to do a two step release or a config
toggle for it.
--
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]
