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

Posted to notifications@geode.apache.org by "mcmellawatt (GitHub)" <gi...@apache.org> on 2019/02/25 21:03:57 UTC

[GitHub] [geode] mcmellawatt commented on pull request #3198: WIP: GEODE-6412: Improve concurrency for getBucketIndex

I assume the hope is that this catch will cover the case of an exception being thrown while iterating over the bucketIndexes synchronized map.  The thing that makes me nervous about this is that the documentation for synchronized maps says:
" It is imperative that the user manually synchronize on the returned map when iterating over any of its collection views
...
**Failure to follow this advice may result in non-deterministic behavior.** "

https://docs.oracle.com/javase/7/docs/api/java/util/Collections.html#synchronizedMap%28java.util.Map%29

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