[
https://issues.apache.org/jira/browse/GEODE-9078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17337660#comment-17337660
]
ASF GitHub Bot commented on GEODE-9078:
---------------------------------------
gaussianrecurrence commented on pull request #776:
URL: https://github.com/apache/geode-native/pull/776#issuecomment-830434263
> @gaussianrecurrence The sub group of people working on geode-native has
historically been a VERY small group and we did a very poor job communicating
outward and recording some of these variations. I wonder if for the sake of not
holding up this PR if you should just blackout that variable renaming commit,
sorry, and we start an open discussion on the dev@geode list on the C++ style
guide. We should discuss deviations we want and how far we should go to try to
refactor code to fit it. Then we should record the consensus in the
CONTRIBUTING.md.
Yes, I also think that would be the fastest approach for this PR. Will
create a new PR with that change and solving Windows compilation
--
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:
[email protected]
> 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)