Issue #16172 has been updated by Andrew  Parker.

Status changed from Unreviewed to Rejected

The data written to disk is a storage format. As such pretty printing is a 
questionable use of time and space on a code path that is supposed to be fast. 
I think if a user wants to read that data, I think it is reasonable that they 
use something to make it more human readable. It looks like the simplest thing 
would be to pipe the file to `python -mjson.tool` (got this snippet from 
<http://stackoverflow.com/questions/352098/how-to-pretty-print-json-script>)
----------------------------------------
Bug #16172: JSON should be pretty-printed when serialized to disk
https://projects.puppetlabs.com/issues/16172#change-70217

Author: Luke Kanies
Status: Rejected
Priority: Normal
Assignee: 
Category: usability
Target version: 
Affected Puppet version: 3.0.0-rc5
Keywords: 
Branch: 


Now that we're storing catalogs as json instead of yaml, it's painful that we 
don't pretty-print them.  You have to go through a complicated process to read 
them at all, and if we just spared some carriage returns, it'd be much better.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to