[ https://issues.apache.org/jira/browse/GEODE-9078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17335782#comment-17335782 ]
ASF GitHub Bot commented on GEODE-9078: --------------------------------------- pivotal-jbarrett commented on pull request #776: URL: https://github.com/apache/geode-native/pull/776#issuecomment-829524591 > ``` > TODO. Review mutex usage in PdxType as it is only used to read, so > either it's not necessary or we are missing write locks. > ``` > > Anyone happens to know why is there a RW mutex to protect just read access? Seems quite odd to me :S I vaguely recall this. I think at the time I didn't have time to research why it only ever appears to acquire the read lock. If it only acquires a read lock then it's rather useless and should be removed. It's also possible that there should be a write lock somewhere. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Remove ACE mutexes > ------------------ > > Key: GEODE-9078 > URL: https://issues.apache.org/jira/browse/GEODE-9078 > Project: Geode > Issue Type: Task > Components: native client > Reporter: Mario Salazar de Torres > Assignee: Mario Salazar de Torres > Priority: Major > Labels: obliterate-ace, pull-request-available > > *AS AN* geode-native contributor > *I WANT TO* remove all occurrences of ACE mutexes > *SO THAT* we can get rid of ACE for good -- This message was sent by Atlassian Jira (v8.3.4#803005)