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

Add 'metadata' option to include metadata fields #23

Open
magnusbaeck opened this issue Apr 17, 2016 · 1 comment · May be fixed by #24
Open

Add 'metadata' option to include metadata fields #23

magnusbaeck opened this issue Apr 17, 2016 · 1 comment · May be fixed by #24
Labels

Comments

@magnusbaeck
Copy link
Contributor

Currently the only way (?) to include the metadata fields in an output is by using the rubydebug codec, which is fine if the consumer is human but if a program is to read the data that's not very useful. I suggest that the json_lines codec gets the same treatment. If this is accepted I'll make the same addition to the json codec.

A case in point is the Logstash Filter Verifier tool which currently can't be used to test Logstash configurations that output events with metadata fields set as is relies on being able to parse the Logstash output via the json_lines codec. Being able to test metadata fields was requested in magnusbaeck/logstash-filter-verifier#5.

@andrewvc
Copy link
Contributor

I think this is a good idea @magnusbaeck !

@andrewvc andrewvc added the P2 label May 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants