virajjasani commented on PR #5754:
URL: https://github.com/apache/hadoop/pull/5754#issuecomment-1615308837

   @mukund-thakur i tried using guava LoadingCache, it's not consistently able 
to evict cache entries, it's doing asynchronously with weak ref and hence 
leading to inconsistent num of entries.
   
   for instance, even when i set max size as 1, i can see 8 entries in the map 
for more than 15s. hence, maintaining consistency with concurrency seems really 
problematic with this implementation.
   there is option to set concurrency too, but still somehow eviction is not 
frequent enough, i suspect this might be because of this:
   ```
   An update to the map and recording
   of reads may not be immediately reflected on the algorithm's data structures.
   ```


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to