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.

Sign up for GitHub

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

Jump to bottom

[APM] Add documentation for APM data and queries #113542

Merged
merged 25 commits into from
Oct 5, 2021
Merged

[APM] Add documentation for APM data and queries #113542

merged 25 commits into from
Oct 5, 2021

Conversation

sorenlouv
Copy link
Member

@sorenlouv sorenlouv commented Sep 30, 2021 •
edited
Loading

Closes #113521

@sorenlouv sorenlouv requested a review from a team as a code owner September 30, 2021 14:33
@botelastic botelastic bot added the Team:APM All issues that need APM UI Team support label Sep 30, 2021
@sorenlouv sorenlouv added release_note:skip Skip the PR/issue when compiling release notes v7.16.0 labels Sep 30, 2021
smith
smith approved these changes Oct 4, 2021
View reviewed changes
Copy link
Contributor

@smith smith left a comment

Choose a reason for hiding this comment

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

Thanks for adding this. Extremely helpful.
tenor-121444434

x-pack/plugins/apm/dev_docs/apm_queries.md

Transactions are stored in two different formats:

#### Individual transactions document
Copy link
Contributor

Choose a reason for hiding this comment

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

Why go from h1 to h4?

x-pack/plugins/apm/dev_docs/apm_queries.md Outdated Show resolved Hide resolved
x-pack/plugins/apm/dev_docs/apm_queries.md Outdated Show resolved Hide resolved
sorenlouv and others added 3 commits October 4, 2021 21:40
@sorenlouv @smith
Update x-pack/plugins/apm/dev_docs/apm_queries.md
dbbcad6 
Co-authored-by: Nathan L Smith <nathan.smith@elastic.co>
@sorenlouv @smith
Update x-pack/plugins/apm/dev_docs/apm_queries.md
94e889a 
Co-authored-by: Nathan L Smith <nathan.smith@elastic.co>
@graphaelli
tidy up transaction breakdown metric sample doc
d06d361
graphaelli
graphaelli approved these changes Oct 5, 2021
View reviewed changes
Copy link
Member

@graphaelli graphaelli left a comment

Choose a reason for hiding this comment

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

I support merging this as is with a goal of moving it over to public docs - Ideally we can cross link this to the corresponding ingestion docs @axw recently added.

@sorenlouv sorenlouv merged commit 853c588 into master Oct 5, 2021
@sorenlouv sorenlouv deleted the apm-data-docs branch October 5, 2021 12:00
@sorenlouv sorenlouv mentioned this pull request Oct 5, 2021
Merged
sorenlouv added a commit that referenced this pull request Oct 5, 2021
@sorenlouv
[APM] Add documentation for APM data and queries (#113542) (#114026)
babfa59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smith smith smith approved these changes

@graphaelli graphaelli graphaelli approved these changes

Assignees
No one assigned
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support v7.16.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[APM] Add documentation for common APM metrics and how to query for them
3 participants
@sorenlouv @smith @graphaelli