You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "jdeppe-pivotal (GitHub)" <gi...@apache.org> on 2018/11/04 04:03:09 UTC

[GitHub] [geode] jdeppe-pivotal commented on issue #2776: GEODE-5982: Synchronized access to CacheLoader and CacheWriter

I don't really know the surrounding context very well, but at face value, I'm not really sure that this change improves anything. For example it seems that 'thread1' could have retrieved the current loader and, before finishing, thread2 changes out the loader and calls `close()` on the old loader before thread1 completes and is done using that old loader.

[ Full content available at: https://github.com/apache/geode/pull/2776 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org