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

[Serverless]: Aggregations #94

Closed
georgewallace opened this issue Oct 21, 2024 · 4 comments
Closed

[Serverless]: Aggregations #94

georgewallace opened this issue Oct 21, 2024 · 4 comments
Assignees
Labels
Serverless bug bash 🐛 Issues discovered during the Serverless UX bug bash

Comments

@georgewallace
Copy link
Contributor

georgewallace commented Oct 21, 2024

Serverless Docs

Welcome to Elastic Serverless

Description

On the Aggregations page:

  1. "Note that scripted metric aggregations are not available in serverless Elasticsearch." should be an actual note and not just in-line text. Could also be under the list as a footnote.
  2. Review the list under "Elasticsearch organizes aggregations into three categories:". Is this list consistent with the other lists across all of our pages?
  3. Under "Aggregation results are in the response's aggregations object:" the example should use []s in hits, this is a property that will return a sub array, while it isn't critical, it lets folks understand that their output in that section could be longer. I.e if it found 17 hits, there would be 17 objects under hits. Technically it is under hits.hits.
  4. Considering the user most likely came from Get Started we should re-think the aggregation examples as the one with time spans uses 1day, and the example on the Get Started page, uses timestamps from over 15 years ago.
  5. In the sentence "The response returns the aggregation type as a prefix to the aggregation's name." we talk about the aggregation types, should we provide a link to the types here?
  6. Under "Aggregation caches" we discuss how things are cached. The links take you to the request_cache docs, when you try to run any of the examples on the stateful docs for it you get an error Request for uri [/books/_stats/request_cache?human] with method [GET] exists but is not available when running in serverless mode

Resources and additional context

This feedback is in regards to Aggregations

@georgewallace georgewallace added the Serverless bug bash 🐛 Issues discovered during the Serverless UX bug bash label Oct 21, 2024
@leemthompo
Copy link
Contributor

This content duplicates https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations.html

Even independent of docs re-migration, I'd advocate for removing duplicate tutorials here and just linking to the original.

Let's see where we get to with our discussions about that re-migration plan.

@leemthompo
Copy link
Contributor

Hey @georgewallace because I've got this aggregations quick start in the pipeline, might be worth you having a run through see what you think?

elastic/elasticsearch#116251

@georgewallace
Copy link
Contributor Author

Agreed. Let's discuss the search and explore your data sections. If we already have the content for each of these we can keep the heading pages and maybe beef them up a bit and point to the source of truth.

@georgewallace
Copy link
Contributor Author

Closing this issue as @leemthompo deeduped the Explore your data section in #202

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Serverless bug bash 🐛 Issues discovered during the Serverless UX bug bash
Projects
None yet
Development

No branches or pull requests

2 participants