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

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2021/03/31 18:40:44 UTC

[GitHub] [geode-native] gaussianrecurrence removed a comment on pull request #776: GEODE-9078: Remove ACE mutexes

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:
users@infra.apache.org