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

Save metrics on close + close only once + check acc registration #296

Merged
merged 12 commits into from
Oct 18, 2016

Commits on Oct 18, 2016

  1. Add ScioOptions registrar

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    1ccbc44 View commit details
    Browse the repository at this point in the history
  2. Refactor metrics

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    07863e4 View commit details
    Browse the repository at this point in the history
  3. Save metrics on close

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    fc57a51 View commit details
    Browse the repository at this point in the history
  4. Add tests for save metrics on close

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    43a48c1 View commit details
    Browse the repository at this point in the history
  5. Remove obsolete import

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    9202956 View commit details
    Browse the repository at this point in the history
  6. Rename pipelineOp to requireNotClosed

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    5b6d549 View commit details
    Browse the repository at this point in the history
  7. Close only once

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    7207f29 View commit details
    Browse the repository at this point in the history
  8. Add test for close context only once

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    7829960 View commit details
    Browse the repository at this point in the history
  9. Require that accumulator is available in the result

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    8dcd381 View commit details
    Browse the repository at this point in the history
  10. Add containsAccumulator to ScioContext

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    8587c5b View commit details
    Browse the repository at this point in the history
  11. Require accumulator to be registered

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    08dd2fc View commit details
    Browse the repository at this point in the history
  12. Add tests for accumulator requirments

    Rafal Wojdyla committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    f98ffba View commit details
    Browse the repository at this point in the history