[ 
https://issues.apache.org/jira/browse/GEODE-9078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17312615#comment-17312615
 ] 

ASF GitHub Bot commented on GEODE-9078:
---------------------------------------

gaussianrecurrence removed a comment on pull request #776:
URL: https://github.com/apache/geode-native/pull/776#issuecomment-811313472


   > @gaussianrecurrence Please rebase onto `develop` and we will review. We 
merged a couple of old outstanding PRs this morning, one of which was _huge_, 
so there are conflicts to resolve.
   
   Will do. However note that there is one thing to look into:
   ```
   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


-- 
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)

Reply via email to