Skip to content
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

Fix the asynchronous worker timeout issue. #3

Merged
merged 1 commit into from
Dec 22, 2019
Merged

Fix the asynchronous worker timeout issue. #3

merged 1 commit into from
Dec 22, 2019

Conversation

whelan9453
Copy link
Owner

CATEGORY

Choose one

  • Bug Fix
  • Enhancement (new features, refinement)
  • Refactor
  • Add tests
  • Build / Development Environment
  • Documentation

SUMMARY

Workers silent for more than this many seconds are killed and restarted. By silent, it means silent from the perspective of the arbiter process, which communicates with the workers through a temporary file. If the worker is busy sending data, it does not update that file. From the perspective of the arbiter, the worker is missing heartbeats.

We add sleep(0) between generator iterations to give the worker a break and update the heartbeat file to keep the connection and worker process alive.

Workers silent for more than this many seconds are killed and restarted. By silent, it means silent from the perspective of the arbiter process, which communicates with the workers through a temporary file. If the worker is busy sending data, it does not update that file. From the perspective of the arbiter, the worker is missing heartbeats.

We add sleep(0) between generator iterations to give the worker a break and update the heartbeat file to keep the connection and worker process alive.
@whelan9453 whelan9453 merged commit c3f1442 into 0.35 Dec 22, 2019
whelan9453 added a commit that referenced this pull request Dec 24, 2019
Provide more information to App Insights
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant