Re: jsonb and nested hstore - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: jsonb and nested hstore
Date
Msg-id 20140227010655.GL4759@eldon.alvh.no-ip.org
Whole thread Raw
In response to Re: jsonb and nested hstore  (Peter Geoghegan <pg@heroku.com>)
Responses Re: jsonb and nested hstore
List pgsql-hackers
Peter Geoghegan wrote:
> On Wed, Feb 26, 2014 at 2:10 PM, Andrew Dunstan <andrew@dunslane.net> wrote:
> > new patch attached, change pushed to github.
> 
> > + /* GUC variables */
> > + static bool    pretty_print_var = false;
> > + #define SET_PRETTY_PRINT_VAR(x)        ((pretty_print_var) ? \
> > +                                      ((x) | PrettyPrint) : (x))
> 
> I think that this is not a great idea. I think that we should do away
> with the GUC, but keep the function hstore_print() so we can pretty
> print that way. I don't believe that this falls afoul of the usual
> obvious reasons for not varying the behavior of IO routines with a
> GUC, since it only varies whitespace, but it is surely pretty
> questionable to have this GUC's setting vary the output of hstore_out,
> an IMMUTABLE function.

I don't see this in the submitted patch.  What's going on?

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services



pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: jsonb and nested hstore
Next
From: Peter Geoghegan
Date:
Subject: Re: jsonb and nested hstore