-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Fix formatting for event.duration
#11675
Conversation
Human readable not working well here.
event.duration
event.duration
Note that I've built one of the Beats, performed its |
The new graph is better then the old one but still not very nice. Also it lacks the unit. I think this is a more general issue Kibana elastic/kibana#20374 I'm good with getting this into Beats for now but not sure if I would make it part of ECS directly as I would hope we could get overall a better solution long term. At the same time I would not consider these changes to be breaking changes so we could add it to ECS and change it later. |
jenkins, test this |
@@ -613,6 +613,8 @@ | |||
type: long | |||
format: duration | |||
input_format: nanoseconds | |||
output_format: asMilliseconds | |||
output_precision: 1 |
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.
Just as a side note, we previously had some confusion around showing durations that are stored and declared as nanoseconds in a different format (as this also changes format in discover), see elastic/apm-server#1717
Human readable not working well for us.
Human readable not working well for us.
Human readable not working well for us.
Human readable not working well for us.
I don't think this is a blocker for 7.0.0. The workaround is simply to edit the Kibana index pattern's formatting options for event.duration.
But this should absolutely go to 7.0.1 if possible.
Note that this is a quick fix applied on a file generated by ECS. If we go forward with this fix, I will backport it to ECS