New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pooled timer fixes #163

Merged
merged 2 commits into from Sep 7, 2017

Conversation

Projects
None yet
2 participants
@gyoetam
Contributor

gyoetam commented Sep 7, 2017

Without these fixes, the pooled timer implementations always reports themselves as not started and the stack fails to schedule transaction timers on them; switching to them simply breaks the stack.

gyoetam added some commits Sep 7, 2017

Fix isStarted() logic for AffinitityExecutorSipTimer
Without this fix, timer always reports itself as not started and the stack fails to schedule transaction timers on it
Fix isStarted() logic for ScheduledExecutorSipTimer
Without this fix, timer always reports itself as not started and the stack fails to schedule transaction timers on it
@jaimecasero

This comment has been minimized.

Collaborator

jaimecasero commented Sep 7, 2017

this bug seems to be there from the original version, and moved to affinity version. I will check the outcome of testsuite

@jaimecasero jaimecasero merged commit f1c952e into RestComm:master Sep 7, 2017

@gyoetam gyoetam deleted the gyoetam:pooled_timer_fix branch Sep 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment