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

[druid] Allow custom druid postaggregators #3146

Merged

Commits on Jul 27, 2017

  1. [druid] Allow custom druid postaggregators

    Also, fix the postaggregation for approxHistogram quantiles so it adds
    the dependent field and that can show up in the graphs/tables.
    
    In general, postAggregators add significant power, we should probably
    support including custom postAggregators. Plywood has standard
    postAggregators here, and a customAggregator escape hatch that allows
    you to define custom postAggregators.
    
    This commit adds a similar capability for Superset and a additional
    field/fields/fieldName breakdown of the typical naming for dependent
    aggregations, which should make it significantly easier to develop
    approxHistogram and custom postAggregation-required dashboards.
    brianwolfe committed Jul 27, 2017
    Configuration menu
    Copy the full SHA
    03be3cf View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    96cfbfe View commit details
    Browse the repository at this point in the history

Commits on Jul 28, 2017

  1. [druid] Apply code review suggestions

    * break out CustomPostAggregator into separate class. This just cleans
      up the creation of the postaggregator a little bit.
    * minor style issues.
    * move the function around so the git diff is more readable
    brianwolfe committed Jul 28, 2017
    Configuration menu
    Copy the full SHA
    9a52e98 View commit details
    Browse the repository at this point in the history