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

Removed aggregations from ReduceContext #10509

Merged
merged 1 commit into from Apr 9, 2015
Merged

Removed aggregations from ReduceContext #10509

merged 1 commit into from Apr 9, 2015

Commits on Apr 9, 2015

  1. Aggregations: removed aggregations from ReduceContext 

    ReduceContext contains the list of aggregations to reduce but these aggregations are set as null half of the time. This change makes the reduce(ReduceContext) method changed to reduce(List<InternalAggregation>, ReduceContext) and ReduceContext now only holds the BigArrays and Script services.
    @colings86
    colings86 committed Apr 9, 2015
    Copy the full SHA
    fcc09f6 View commit details
    Browse the repository at this point in the history