Optional JSON indent to allow storing data as json-lines

[dimaqq]

Description

Would the OTEL project be open to making JSON identation optional?

That would allow, for example, dumping spans into jsonl files.

Type of change

Please delete options that are not relevant.

• Relaxation of type hints (minor feature)

How Has This Been Tested?

• Tested locally, can post code later

Contrib Repo Change?

no

Checklist:

Not yet, let's discuss this first.

• Followed the style guidelines of this project
• Changelogs have been updated
• Unit tests have been added
• Documentation has been updated

P.S. I'm happy to contribute this change throughout the codebase (also logging, metrics, unit tests, etc.)

[linux-foundation-easycla]

• ❌ login: @dimaqq / The commit (7ac5f97). This user is authorized, but they must confirm their affiliation with their company. Start the authorization process by clicking here, click "Corporate", select the appropriate company from the list, then confirm your affiliation on the page that appears. For further assistance with EasyCLA, please submit a support request ticket.

[dimaqq]

P.S. just in case it's not clear, indent=None and indent=0 are not the same in json.dumps.

[dimaqq]

On the second thought, I'm not sure if this is worth it.