adding sum aggregations docs #10256
adding sum aggregations docs #10256
Conversation
Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
AntonEliatra
requested review from
kolchfa-aws,
AMoo-Miki,
natebower,
dlvenable,
epugh and
sumobrian
as code owners
|
Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged. Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer. When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review. |
Signed-off-by: Anton Rubin <anton.rubin@eliatra.com>
|
@bowenlan-amzn Could you please review this PR? Thanks! |
kolchfa-aws
added
Tech review
| "total_weight_with_missing": { | ||
| "sum": { | ||
| "field": "weight_kg", | ||
| "missing": 0 |
There was a problem hiding this comment.
Maybe we don't need an example, but a table of parameter supported.
There was a problem hiding this comment.
+1: @AntonEliatra Could you please add a parameter table for this page (and any aggregations page that doesn't have it)? See for example https://docs.opensearch.org/docs/latest/aggregations/bucket/geohash-grid/#supported-parameters
| "aggs": { | ||
| "adjusted_weight": { | ||
| "sum": { | ||
| "field": "weight_kg", |
There was a problem hiding this comment.
Feels pretty similar to the previous example, can we combine them?
Description
adding sum aggregations docs
Version
all
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.