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

i#6938 migrate: Include sched stats in query and fix related assert #7060

Merged
merged 1 commit into from
Oct 29, 2024

Commits on Oct 29, 2024

  1. i#6938 migrate: Include sched stats in query and fix related assert

    schedule_stats_t::get_total_counts() was not including
    scheduler-provided stats, as it was doing its own simple aggregation
    instead of calling aggregate_results().  We fix that here.  That then
    triggers the newly added assert from PR #7057 which checks for the
    scheduler-provided value being exactly equal meaning there is no data
    available.  It fires on the schedule_stats_test, which uses a mock
    stream which returns -1 for such a stat, so we end up with a negative
    value.  We update the assert for that condition.
    
    Issue: #6938
    derekbruening committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    41f1184 View commit details
    Browse the repository at this point in the history