-
Notifications
You must be signed in to change notification settings - Fork 417
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 JSON formatting to OStreamExporters #202
Comments
This issue was marked as stale due to lack of activity. It will be closed in 7 days if no furthur activity occurs. |
I started working on this in #1111. |
This issue was marked as stale due to lack of activity. It will be closed in 7 days if no furthur activity occurs. |
Closed as inactive. Feel free to reopen if this is still an issue. |
Does Otel support the JSON output now? @lalitb @deejgregor |
Not yet, I create #2482 to track this. |
Is your feature request related to a problem?
There is currently no way for the user to change the format that OStreamExporters export the data. If the user wants to send their data directly to a json file, it will not be in the correct format.
Describe the solution you'd like
Have the user pass in a boolean for json printing in the constructor. Then in export() format the data what the user has specified. Since the user won't directly call export() it makes sense to pass the boolean in the constructor.
Describe alternatives you've considered
An alternative is to support json formatting as the default and only printing. This doesn't seem ideal as most users will only print to stdout and will have a harder time reading the output.
Additional context
The two exporters are 218 and 183
The text was updated successfully, but these errors were encountered: