-
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
Add jvm9 logging format to GC fileset of ES module #7390
Conversation
After `date` processor has been added @timestamp is of type date so the its formatting will be different
@@ -14,3 +14,11 @@ | |||
type: float | |||
description: > | |||
Time took to stop threads seconds. | |||
- name: pid |
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.
I wonder if we should change this to process.pid
in a follow up PR? https://github.com/elastic/ecs#process
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.
sounds good. tags
field seems like a good candidate too.
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.
initially I thought to rename elasticsearch.gc.tags
to tags
but now I don't think it's a good idea. Tags
are more generic while elasticsearch.gc.tags
are GC specific fields with values of safepoint
, gc
, heap
, age
. I will rename PID field only.
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.
++ on what you stated about the tags
. We had very similar discussions in other places like docker
and often these tags are service specific so don't belong in the global ones.
follow-up of elastic#7390
* rename PID field name to match ECS scheme * Add darwin and windows paths to the log files * simplify windows path pattern follow-up of #7390
No description provided.