-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
[DOCS] Correct percentile rank agg example responses #50052
[DOCS] Correct percentile rank agg example responses #50052
Conversation
Pinging @elastic/es-docs (>docs) |
Pinging @elastic/es-analytics-geo (:Analytics/Aggregations) |
The example snippets in the percentile rank agg docs use a test dataset named latency, which is generated from docs/gradle.build. At some point the dataset and example snippets were updated, but the text surrounding the snippets was not. This meant the text and the example snippets shown no longer match up. Because the surrounding text seemed more realistic, this changes the snippets to match using //TESTRESPONSE magic comments.
@elasticmachine update branch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I think this trickery is fine. A lot easier than trying to craft a new dataset that fits the narrative, and still tests what we want. I like it!
Thanks for fixing this <3
The example snippets in the percentile rank agg docs use a test dataset named `latency`, which is generated from docs/gradle.build. At some point the dataset and example snippets were updated, but the text surrounding the snippets was not. This means the text and the example snippets shown no longer match up. This corrects that by changing the snippets using /TESTRESPONSE magic comments.
The example snippets in the percentile rank agg docs use a test dataset named `latency`, which is generated from docs/gradle.build. At some point the dataset and example snippets were updated, but the text surrounding the snippets was not. This means the text and the example snippets shown no longer match up. This corrects that by changing the snippets using /TESTRESPONSE magic comments.
The example snippets in the percentile rank agg docs use a test dataset named `latency`, which is generated from docs/gradle.build. At some point the dataset and example snippets were updated, but the text surrounding the snippets was not. This means the text and the example snippets shown no longer match up. This corrects that by changing the snippets using /TESTRESPONSE magic comments.
The example snippets in the percentile rank agg docs use a test dataset named `latency`, which is generated from docs/gradle.build. At some point the dataset and example snippets were updated, but the text surrounding the snippets was not. This means the text and the example snippets shown no longer match up. This corrects that by changing the snippets using /TESTRESPONSE magic comments.
The example snippets in the percentile rank agg docs use a test dataset named
latency
, which is generated from docs/gradle.build.At some point the dataset and example snippets were updated, but the text surrounding the snippets was not. This meant the text and the example snippets shown no longer match up.
Because the surrounding text seemed more realistic, this changes the snippets to match using //TESTRESPONSE magic comments.
Let me know if we prefer to create a new example dataset. This just seemed like a simpler, more straightforward solution.
Supersedes/closes #45069, #45985, #48121