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

Add fields for source #3

Closed
felixbarny opened this issue Aug 13, 2019 · 2 comments · Fixed by #31
Closed

Add fields for source #3

felixbarny opened this issue Aug 13, 2019 · 2 comments · Fixed by #31

Comments

@felixbarny
Copy link
Member

Map the source of the log event (class, function, line number) to ECS fields. Currently, there are no appropriate fields in ECS.

See also the discussion in elastic/ecs#281 (comment)
@felixbarny
Copy link
Member Author

Adding these fields should be optional and off by default. It adds both runtime and disk overhead to gather and store that information for each log event.

@felixbarny
Copy link
Member Author

I have created elastic/ecs#563 for that

felixbarny added a commit to felixbarny/java-ecs-logging that referenced this issue Sep 17, 2019
@felixbarny
Add includeOrigin flag
e1e796d 
depends on elastic/ecs#563
closes elastic#3
felixbarny added a commit to felixbarny/java-ecs-logging that referenced this issue Sep 17, 2019
@felixbarny
Add includeOrigin flag
1df2559 
depends on elastic/ecs#563
closes elastic#3
@felixbarny felixbarny mentioned this issue Sep 17, 2019
Closed
felixbarny added a commit that referenced this issue Sep 18, 2019
@felixbarny
Add includeOrigin flag
83b2543 
depends on elastic/ecs#563
closes #3
felixbarny added a commit to felixbarny/java-ecs-logging that referenced this issue Sep 19, 2019
@felixbarny
Add includeOrigin flag
3230f31 
depends on elastic/ecs#563
closes elastic#3
@felixbarny felixbarny mentioned this issue Sep 19, 2019
Merged
felixbarny added a commit that referenced this issue Sep 19, 2019
@felixbarny
Add includeOrigin flag (#31)
74e11f0 
depends on elastic/ecs#563
closes #3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add includeOrigin flag felixbarny/java-ecs-logging
1 participant
@felixbarny