Add children aggregation documentation #7645
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Melissa Vagi <[email protected]>
vagimeli
requested review from
hdhalter,
kolchfa-aws,
Naarcha-AWS,
AMoo-Miki,
natebower,
dlvenable,
stephen-crawford and
epugh
as code owners
11 tasks
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
vagimeli
commented
Jul 8, 2024
_aggregations/bucket/children.md
Outdated
|
|
||
| The `authors` aggregation groups the documents by the `author.keyword` field. Within each author group, we have a `children` aggregation that looks at the associated posts. Inside the `posts` aggregation, another `children` aggregation looks at the comments associated with each post. Within the `comments` aggregation, the `value_count` aggregation counts the number of comments for each post. | ||
|
|
||
| #### Example |
There was a problem hiding this comment.
Technical reviewer: Please confirm this example is relevant to an OpenSearch user. I tested the example using Dev Tools. If another example is more appropriate, please replace the draft example with your example. Thank you.
There was a problem hiding this comment.
@msfroh @bowenlan-amzn @sandeshkr419 Would one of you be available to technical review this PR? The doc team is closing content gaps for aggregations and need a technical review before we can published the doc. Please note that I used internal tools to assist in code example generation and Dev Tools to test those examples. Thank you
There was a problem hiding this comment.
I only see the search request and response here, can you provide what data you ingest?
There was a problem hiding this comment.
@bowenlan-amzn I used the sample data sets in OpenSearch Dashboards. Feel free to update with relevant examples for OpenSearch users.
There was a problem hiding this comment.
Using the sample data sets is fine, I think just mention that here and this is good.
Signed-off-by: Melissa Vagi <[email protected]>
vagimeli
commented
Oct 8, 2024
_aggregations/bucket/children.md
Outdated
|
|
||
| The `children` aggregation connects parent documents with their related child documents. This allows you to analyze relationships between different types of data in a single query, rather than having to run multiple queries and combine the results yourself. | ||
|
|
||
| For example, if you have a parent-child relationship between authors, posts, and comments, you can analyze the relationships between the data different types (`authors`, `posts`, and `comments`) in a single query, without having to run multiple queries and combine the results manually. |
There was a problem hiding this comment.
@bowenlan-amzn Does this clarify the sample index and data comment?
There was a problem hiding this comment.
I think you need to provide the requests you used to index the data that correspond to below search request and response.
Signed-off-by: Melissa Vagi <[email protected]>
vagimeli
added
4 - Doc review
natebower
reviewed
Oct 8, 2024
| For example, if you have a parent-child relationship between authors, posts, and comments, you can analyze the relationships between the data different types (`authors`, `posts`, and `comments`) in a single query, without having to run multiple queries and combine the results manually. | ||
|
|
||
| The `authors` aggregation groups the documents by the `author.keyword` field. Within each author group, we have a `children` aggregation that looks at the associated posts. Inside the `posts` aggregation, another `children` aggregation looks at the comments associated with each post. Within the `comments` aggregation, the `value_count` aggregation counts the number of comments for each post. | ||
|
|
There was a problem hiding this comment.
Above: Please replace both instances of the phrasal verb "looks at" with a more precise verb. What do the aggregations actually do?
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
vagimeli
commented
Oct 11, 2024
vagimeli
commented
Oct 11, 2024
Signed-off-by: Melissa Vagi <[email protected]>
vagimeli
commented
Oct 11, 2024
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
leanneeliatra
pushed a commit
to leanneeliatra/opensearch-documentation-website-forl
that referenced
this pull request
Oct 17, 2024
* Add children aggregation documentation Signed-off-by: Melissa Vagi <[email protected]> * Add children aggregation documentation Signed-off-by: Melissa Vagi <[email protected]> * Add children aggregation documentation Signed-off-by: Melissa Vagi <[email protected]> * Add content gaps to list Signed-off-by: Melissa Vagi <[email protected]> * Update examples to include data samples Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/index.md Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/index.md Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Signed-off-by: Melissa Vagi <[email protected]> * Fix dead links Signed-off-by: Melissa Vagi <[email protected]> --------- Signed-off-by: Melissa Vagi <[email protected]> Co-authored-by: Naarcha-AWS <[email protected]> Co-authored-by: Nathan Bower <[email protected]>
leanneeliatra
pushed a commit
to leanneeliatra/opensearch-documentation-website-forl
that referenced
this pull request
Oct 17, 2024
* Add children aggregation documentation Signed-off-by: Melissa Vagi <[email protected]> * Add children aggregation documentation Signed-off-by: Melissa Vagi <[email protected]> * Add children aggregation documentation Signed-off-by: Melissa Vagi <[email protected]> * Add content gaps to list Signed-off-by: Melissa Vagi <[email protected]> * Update examples to include data samples Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/index.md Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/index.md Signed-off-by: Melissa Vagi <[email protected]> * Update _aggregations/bucket/children.md Signed-off-by: Melissa Vagi <[email protected]> * Fix dead links Signed-off-by: Melissa Vagi <[email protected]> --------- Signed-off-by: Melissa Vagi <[email protected]> Co-authored-by: Naarcha-AWS <[email protected]> Co-authored-by: Nathan Bower <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Closes content gaps for the following bucket aggregations:
Auto-interval date histogram
Children
Composite
Parent
Rare terms
Issues Resolved
#4293
Version
Current version
Frontend features
If you're submitting documentation for an OpenSearch Dashboards feature, add a video that shows how a user will interact with the UI step by step. A voiceover is optional.
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.