[GitHub] [ozone] GeorgeJahad commented on pull request #4567: HDDS-8528. [Snapshot] Custom SnapshotCache implementation to replace LoadingCache

["GeorgeJahad (via GitHub)" <gi...@apache.org>]

GeorgeJahad commented on PR #4567:
URL: https://github.com/apache/ozone/pull/4567#issuecomment-1540696308

   @smengcl I took a quick look at ReferenceCounted.java and SnapshotCache.java.  They look good to me.  (The cleanup() method probably requires some synchronization.)  I'll take a closer look when the PR comes out of draft state.
   


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org