[
https://issues.apache.org/jira/browse/GEODE-9056?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17308039#comment-17308039
]
ASF GitHub Bot commented on GEODE-9056:
---------------------------------------
gaussianrecurrence commented on a change in pull request #772:
URL: https://github.com/apache/geode-native/pull/772#discussion_r600704330
##########
File path: cppcache/src/TcrConnectionManager.cpp
##########
@@ -72,7 +72,7 @@ void TcrConnectionManager::init(bool isPool) {
}
auto &props = m_cache->getDistributedSystem().getSystemProperties();
m_isDurable = !props.durableClientId().empty();
- auto pingInterval = (props.pingInterval() / 2);
+ auto pingInterval = props.pingInterval();
Review comment:
This is something interesting hehe. Thing is previous implementation
called twice to semaphore acquire. The implementation was clearly wrong. But in
order to solve the issue the ping interval was divided by 2, et voilà.
So now the right interval is used and ping semaphore acquire is just called
once.
--
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]
> Replace ACE_Semaphore by an alternative
> ---------------------------------------
>
> Key: GEODE-9056
> URL: https://issues.apache.org/jira/browse/GEODE-9056
> 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 A* geode-native contributor
> *I WANT* to replace ACE_Semaphore by any non-ACE alternative
> *SO THAT* we can get rid of ACE any time soon.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
