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

[APM] Add missing trace_id to transaction histogram #27591

Merged
merged 4 commits into from
Dec 21, 2018

Conversation

sorenlouv
Copy link
Member

@sorenlouv sorenlouv commented Dec 20, 2018

transaction_id is not globally unique but only unique within a given trace. To avoid collisions at scale it is therefore necessary to query transactions by both trace_id and transaction_id.

Additional changes:

  • When a transaction was not found the API should respond with 404, and client should handle this
  • Fix an unrelated bug with StickyPropertiesTable where boolean values are not visible (must be stringified)

This will not be backported to 6.6.

@sorenlouv sorenlouv requested a review from a team as a code owner December 20, 2018 15:56
@sorenlouv sorenlouv added the Team:APM All issues that need APM UI Team support label Dec 20, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@elasticmachine
Copy link
Contributor

💔 Build Failed

@sorenlouv
Copy link
Member Author

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@sorenlouv
Copy link
Member Author

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@sorenlouv sorenlouv merged commit c7bd296 into elastic:master Dec 21, 2018
@sorenlouv sorenlouv deleted the add-missing-trace-id branch December 21, 2018 13:21
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request Dec 27, 2018
* [APM] Add missing trace_id to transaction histogram

* Handle 404 properly

* Fix `unhandled` issue

* Fix tests
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request Dec 27, 2018
sorenlouv added a commit that referenced this pull request Dec 28, 2018
…[APM] StickyProperties should render components as values (#27814) | Revert changes to Monitoring accidentally introduced in #27591 (#278 (#27836)

* [APM] Add missing trace_id to transaction histogram (#27591)

* [APM] Add missing trace_id to transaction histogram

* Handle 404 properly

* Fix `unhandled` issue

* Fix tests

* [APM] StickyProperties should render components as values (#27814)

* Revert changes to Monitoring accidentally introduced in #27591 (#27830)
clintandrewhall pushed a commit to clintandrewhall/kibana that referenced this pull request Jan 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:APM All issues that need APM UI Team support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants