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

Unify histogram implementations #9446

Merged
merged 1 commit into from Jan 28, 2015
Merged

Unify histogram implementations #9446

merged 1 commit into from Jan 28, 2015

Commits on Jan 28, 2015

  1. Aggregations: Unify histogram implementations 

    This change makes InternalHistogram the only InternalAggregation used by the Histogram Aggregator. There is still a separate Bucket implementation and Factory implementation. All buckets are created through the factory passed into the InternalHistogram meaning and the correct factory implementation is serialised as part of the aggregation to make sure the correct bucket types are always generate.

This is needed by the Transformers (namely the derivative transformer) to allow it to generate buckets of the right type without having to know what the underlying bucket implementation is.
    @colings86
    colings86 committed Jan 28, 2015
    Copy the full SHA
    29c24d7 View commit details
    Browse the repository at this point in the history