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

Call ensureNoSelfReferences() on _agg state variable after scripted metric agg script executions #31044

Merged
merged 2 commits into from
Jun 11, 2018
Merged

Call ensureNoSelfReferences() on _agg state variable after scripted metric agg script executions #31044

merged 2 commits into from
Jun 11, 2018

Commits on Jun 2, 2018

  1. Call ensureNoSelfReferences() on _agg state variable after scripted m… 

    …etric agg script executions

Previously this was called for the combine script only. This change checks for self references for
init, map, and reduce scripts as well, and adds unit test coverage for the init, map, and combine cases.
    @rationull
    rationull committed Jun 2, 2018
    Configuration menu
    Copy the full SHA
    8b9a44f View commit details
    Browse the repository at this point in the history

Commits on Jun 8, 2018

  1. Remove CollectionUtil.ensureNoSelfReference() overload without a mess… 

    …ageHint per PR feedback
    @rationull
    rationull committed Jun 8, 2018
    Configuration menu
    Copy the full SHA
    34b9e4b View commit details
    Browse the repository at this point in the history