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

Investigate to move transaction.span_count.dropped.total to transaction.span_count.dropped #1675

Closed
simitt opened this issue Dec 19, 2018 · 2 comments · Fixed by #1809
Closed

Investigate to move transaction.span_count.dropped.total to transaction.span_count.dropped #1675

simitt opened this issue Dec 19, 2018 · 2 comments · Fixed by #1809
Assignees
@graphaelli
Labels
breaking change
Milestone
7.0

Comments

@simitt
Copy link
Contributor

simitt commented Dec 19, 2018

Move transaction.span_count.dropped.total to transaction.span_count.dropped so it's at the same level as transaction.span_count.started and aligns with the v2 intake protocol.

I don't see how we can change that without conflicting with 6.x indices, as the type of the ES field would change.
@simitt simitt added this to the 7.0 milestone Dec 19, 2018
@roncohen roncohen mentioned this issue Dec 19, 2018
Closed
6 tasks
@jalvz
Copy link
Contributor

jalvz commented Jan 17, 2019

This can be done with the reindexing script

@simitt simitt mentioned this issue Jan 17, 2019
Closed
4 tasks
@graphaelli
Copy link
Member

FYI @elastic/apm-ui, field rename not related to ECS

@graphaelli graphaelli self-assigned this Jan 17, 2019
@graphaelli graphaelli mentioned this issue Jan 17, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@graphaelli graphaelli

Labels
breaking change
Projects
None yet
Milestone
7.0
Development

Successfully merging a pull request may close this issue.

Rename transaction span dropped total graphaelli/apm-server
4 participants
@graphaelli @jalvz @simitt @alvarolobato