Skip to content

Conversation

@marciw
Copy link
Contributor

@marciw marciw commented Oct 13, 2025

(Replaces #3222, which was an ominshambles)

This PR restructures and edits time series data stream docs as part of #1502.

❗ Notes for reviewers:

  • Changes were approved by Product and Engineering in Edit time series docs for clarity #3222 🙈
  • The content is "MVP" quality overall. More improvements coming soon via the follow-up issue, including reconciling the time series section with recent similar changes to the general data stream docs
  • The various applies_to thingies could probably be better; I am committed to maintaining the thinnest of grasps on applies_to metadata

Summary of changes:

@marciw marciw mentioned this pull request Oct 13, 2025
2 tasks
@marciw marciw marked this pull request as ready for review October 13, 2025 21:51
@marciw marciw requested review from a team as code owners October 13, 2025 21:51
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note for reviewers: The OTLP page was added in #3360 (written by @felixbarny)

Slated for more improvements in #3179

@florent-leborgne
Copy link
Contributor

I really like seeing advanced topics explicitly gathered together 👏

Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🦕
This looks great @marciw! The new layout is a big improvement.

Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💅 I had to dig very deep for the nittiest of nits :)

@marciw marciw merged commit 3cd526d into main Oct 17, 2025
8 checks passed
@marciw marciw deleted the mw-tsds-fin branch October 17, 2025 16:58
rhr323 pushed a commit to rhr323/docs-content that referenced this pull request Oct 27, 2025
(Replaces elastic#3222, which was
an ominshambles)

This PR restructures and edits time series data stream docs as part of
elastic#1502.

❗ Notes for reviewers:
- Changes were approved by Product and Engineering in
elastic#3222 🙈
- The content is "MVP" quality overall. More improvements coming soon
via the [follow-up
issue](elastic#3179), including
reconciling the time series section with recent similar changes to the
general data stream docs
- The various `applies_to` thingies could probably be better; I am
committed to maintaining the thinnest of grasps on `applies_to` metadata

Summary of changes:
- Accuracy updates by SME reviewers/contributors
- Many edits for clarity, conciseness, and user focus
- Mentions of new ES|QL TS command (which was one of the primary
motivators for cleaning up this section)
- See also elastic#2274 and
elastic#2742

---------

Co-authored-by: David Kilfoyle <[email protected]>
Co-authored-by: Liam Thompson <[email protected]>
naemono pushed a commit to naemono/docs-content that referenced this pull request Oct 28, 2025
(Replaces elastic#3222, which was
an ominshambles)

This PR restructures and edits time series data stream docs as part of
elastic#1502.

❗ Notes for reviewers:
- Changes were approved by Product and Engineering in
elastic#3222 🙈
- The content is "MVP" quality overall. More improvements coming soon
via the [follow-up
issue](elastic#3179), including
reconciling the time series section with recent similar changes to the
general data stream docs
- The various `applies_to` thingies could probably be better; I am
committed to maintaining the thinnest of grasps on `applies_to` metadata

Summary of changes:
- Accuracy updates by SME reviewers/contributors
- Many edits for clarity, conciseness, and user focus
- Mentions of new ES|QL TS command (which was one of the primary
motivators for cleaning up this section)
- See also elastic#2274 and
elastic#2742

---------

Co-authored-by: David Kilfoyle <[email protected]>
Co-authored-by: Liam Thompson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants