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

Initialize schedule lazily #801

Merged
merged 1 commit into from
Oct 18, 2019

Commits on Oct 18, 2019

  1. Initialize schedule lazily

    Previously, we have initialized the schedule eagerly when it was
    created. Among the state of the schedule is the point in time when it
    was started. However, as the schedule is not immediately executed after
    creation, it is possible that the start timestamp is slightly off. This
    is usually not a problem in practice but may cause test failures where
    we use very short-lived schedules (one second in total).
    
    With this commit we wrap the schedule (which is a generator) in a
    handle. This handle is passed to the executor which can then lazily
    initialize the schedule so the timestamp matches exactly the point in
    time when the schedule is executed.
    
    Closes elastic#600
    danielmitterdorfer committed Oct 18, 2019
    Configuration menu
    Copy the full SHA
    0c5c11e View commit details
    Browse the repository at this point in the history