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

[WIP] Scheduler to replace chronograph #5094

Closed
wants to merge 7 commits into from

Conversation

benjaoming
Copy link
Contributor

Summary

Replaces PR #4907

This is the initial framework for the scheduler, opening up the PR to let the rest of the team know what my thoughts were.

Terminology: Not sure if it should be named a scheduler, it might be more of a producer-consumer pattern, suggestions welcome.

TODO

If not all TODOs are marked, this PR is considered WIP (work in progress)

  • Have tests been written for the new code? If you're fixing a bug, write a regression test (or have a really good reason for not writing one... and I mean really good!)
  • Has documentation been written/updated?
  • New dependencies (if any) added to requirements file
  • Add an entry to CHANGELOG.rst

Reviewer guidance

Comment on anything you like :)

Issues addressed

#4892

@benjaoming
Copy link
Contributor Author

Could have been great :)

@benjaoming benjaoming closed this Sep 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant