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

Avoid respawning task trackers constantly when they are idle #59

Merged
merged 2 commits into from
May 20, 2015

Conversation

tarnfeld
Copy link
Member

Previously the "idle check" would be run against all task trackers regardless of whether they have any jobs assigned to them or not. The main MesosScheduler is responsible for cleaning up task trackers once jobs have finished so this change stops us performing idle checks on trackers that have no jobs.

Destruction of those task trackers will be handled elsewhere.

This change fixes the observed behaviour of task trackers being killed and respawning continuously when they're waiting for jobs to be scheduled on them (e.g with the min map/reduce slot config option, or the fixed resource policy). Essentially this fixed a case when trackers were killed due to being idle, when they were supposed to be sitting idle.

tarnfeld added 2 commits May 19, 2015 10:03
Previously the "idle check" would be run against all task trackers
regardless of whether they have any jobs assigned to them or not. The
main MesosScheduler is responsible for cleaning up task trackers
once jobs have *finished* so this change stops us performing
idle checks on trackers that have no jobs.

This change fixes the observed behaviour of task trackers being killed
and respawning continuously when they're waiting for jobs (e.g with
the min map/reduce slot config option, or the fixed resource policy).
tarnfeld added a commit that referenced this pull request May 20, 2015
Avoid respawning task trackers constantly when they are idle
@tarnfeld tarnfeld merged commit b7a8278 into mesos:master May 20, 2015
@tarnfeld tarnfeld deleted the fix/tt-respawning branch May 20, 2015 11:40
@tarnfeld
Copy link
Member Author

Note that this was reverted from master

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