Skip to content

[analytics-plugin] Dsl percentiles agg - #22596

Open
sachin-27 wants to merge 6 commits into
opensearch-project:mainfrom
sachin-27:dsl-percentiles-agg
Open

[analytics-plugin] Dsl percentiles agg#22596
sachin-27 wants to merge 6 commits into
opensearch-project:mainfrom
sachin-27:dsl-percentiles-agg

Conversation

@sachin-27

Copy link
Copy Markdown

Description

Adds support for the percentiles metric aggregation (t-digest) to the DSL Calcite plugin.
Stacks on #22561 — only the last commit is new to this PR.

Key changes:

• PercentilesMetricTranslator: translates percentiles to PERCENTILE_APPROX aggregate calls (one per requested percent), honoring percents, missing, and format
• t-digest compression via a three-argument PERCENTILE_APPROX_N form, registered across the analytics framework function enum, the engine's literal-argument planning rule, and the DataFusion backend (rewriter + function catalog)
• InternalDslPercentiles: renders the classic tdigest_percentiles response shape, including typed_keys
• hdr method is rejected at conversion (analytics path supports t-digest only)

Check List

  • [x ] Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet