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

Reconcile child summaries correctly #5205

Merged
merged 7 commits into from
Jan 19, 2019
Merged

Reconcile child summaries correctly #5205

merged 7 commits into from
Jan 19, 2019

Commits on Jan 17, 2019

  1. Configuration menu
    Copy the full SHA
    7f4f55b View commit details
    Browse the repository at this point in the history
  2. Fix bug in reconcile summaries that affects periodic/parameterized jobs

    This fixes incorrect parent job summaries by recomputing them in the
    ReconcileJobSummaries method in the state store
    preetapan committed Jan 17, 2019
    Configuration menu
    Copy the full SHA
    b038333 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    9451812 View commit details
    Browse the repository at this point in the history
  4. Refactor to find jobs with child instances more effeciently

    also added unit tests
    preetapan committed Jan 17, 2019
    Configuration menu
    Copy the full SHA
    be32301 View commit details
    Browse the repository at this point in the history
  5. fix linting

    preetapan committed Jan 17, 2019
    Configuration menu
    Copy the full SHA
    946d477 View commit details
    Browse the repository at this point in the history

Commits on Jan 18, 2019

  1. revert unintended change

    preetapan committed Jan 18, 2019
    Configuration menu
    Copy the full SHA
    e58fcf7 View commit details
    Browse the repository at this point in the history
  2. code review comments

    preetapan committed Jan 18, 2019
    Configuration menu
    Copy the full SHA
    b403c9b View commit details
    Browse the repository at this point in the history