Repository navigation
OAK-12425: opt into Caffeine SYNC without metadata changes - #3176
rishabhdaim wants to merge 1 commit into
Conversation
Benchmark re-run after the eviction-listener fixThe results were re-run against Summary:
|
d3616e1 to
be45245
Compare
thomasmueller
left a comment
There was a problem hiding this comment.
I think a document store team expert needs to review the PR.
Adding the value to the MetadataEntry sounds wrong to me.
Increasing the cache size sounds dangerous (risk of OOME).
cache size change has been reverted. |
816417e to
28eadd0
Compare
|
49616cb to
9d097f9
Compare
9ae37dd to
9576c56
Compare
4d44309 to
2625d10
Compare
Co-Authored-By: OpenAI Codex <noreply@openai.com>




Allow DocumentNodeStore caches, including LOCAL_DIFF, to opt into Caffeine SYNC through disabled-by-default
FT_CAFFEINE_CACHE_OAK-12425or-Doak.documentMK.caffeineCache=true.Keep the existing persistent wrapper, metadata, raw values, weights and file format. Feature selection is fixed per builder; changes require recreating the builder/store or restarting the service. Optional ASYNC support follows in #3190.
Document the legacy
guavaCacheproperty and verify that throwing SYNC eviction listeners do not fail cache operations.Validation with Java 17: 52 targeted tests passed; all four system-property combinations verified in fresh JVMs; baseline and license checks passed.
OAK-12425. Depends on #3186. Benchmark results.
Co-Authored-By: OpenAI Codex noreply@openai.com