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.

Sign up for GitHub

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

Jump to bottom

executor could take more than once incorrectly #383

Merged
merged 7 commits into from
Jun 19, 2018
Merged

executor could take more than once incorrectly #383

merged 7 commits into from
Jun 19, 2018

Commits on Jun 19, 2018

  1. Baseline test and force threads to yield.

    @mjcarroll
    mjcarroll committed Jun 19, 2018
    Configuration menu
    Copy the full SHA
    7d80613 View commit details
    Browse the repository at this point in the history

  2. Add timer tracking for executor.

    @mjcarroll
    mjcarroll committed Jun 19, 2018
    Configuration menu
    Copy the full SHA
    6853019 View commit details
    Browse the repository at this point in the history

  3. Add locking and test happy-path exit conditions.

    @mjcarroll
    mjcarroll committed Jun 19, 2018
    Configuration menu
    Copy the full SHA
    a2c7e9e View commit details
    Browse the repository at this point in the history

  4. Move logic to multi_threaded_executor

    @mjcarroll
    mjcarroll committed Jun 19, 2018
    Configuration menu
    Copy the full SHA
    38c48d1 View commit details
    Browse the repository at this point in the history

  5. Address reviewer feedback by reducing scope of set.

    @mjcarroll
    mjcarroll committed Jun 19, 2018
    Configuration menu
    Copy the full SHA
    d51bf80 View commit details
    Browse the repository at this point in the history

  6. Expand tolerance on testing.

    @mjcarroll
    mjcarroll committed Jun 19, 2018
    Configuration menu
    Copy the full SHA
    fdcfd01 View commit details
    Browse the repository at this point in the history

  7. comment fixup 

    Otherwise it seemed to me like it would yield twice.
    @dhood
    dhood authored Jun 19, 2018
    Configuration menu
    Copy the full SHA
    1b5d3fd View commit details
    Browse the repository at this point in the history