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

Posted to notifications@geode.apache.org by "pivotal-jbarrett (GitHub)" <gi...@apache.org> on 2019/09/25 04:20:44 UTC

[GitHub] [geode] pivotal-jbarrett commented on pull request #4085: GEODE-6927 make getThreadOwnedConnection code thread safe

`threadConnectionMap` should probably be made volatile as well. 

I am also not a fan of the `cm` variable name. I would prefer `Map threadConnectionMap = this.threadConnectionMap`. Or something more descriptive than `cm`.

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