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

update utils.py to allow for jsonl dump instead of pretty json. #7051

Open
r32rtb opened this issue Jan 10, 2022 · 3 comments
Open

update utils.py to allow for jsonl dump instead of pretty json. #7051

r32rtb opened this issue Jan 10, 2022 · 3 comments

Comments

@r32rtb
Copy link

r32rtb commented Jan 10, 2022

Describe the feature

The json output for c7n-org reports being pretty json doesn't lend itself to being consumed programmatically. Additional steps are needed to convert format to jsonl. Allow for an output option to output as jsonl so the output can be utilized by other facilities programmatically.

Extra information or context

No response

@kapilt
Copy link
Collaborator

kapilt commented Jan 11, 2022

I'm a little unclear what you mean when you say json can't be loaded programmatically... what are you using to process json lines format?

@kapilt
Copy link
Collaborator

kapilt commented Jan 12, 2022

I found this overview of the different format names for essentially the same thing somewhat helpful
ndjson/ndjson.github.io#1 (comment)

@herrsergio
Copy link

It should be useful to support a jsonl output.
See: https://aws.amazon.com/premiumsupport/knowledge-center/error-json-athena/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants