From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Peter Geoghegan <pg(at)heroku(dot)com> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Andres Freund <andres(at)2ndquadrant(dot)com>, Teodor Sigaev <teodor(at)sigaev(dot)ru>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: jsonb and nested hstore |
Date: | 2014-02-27 01:06:55 |
Message-ID: | 20140227010655.GL4759@eldon.alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Peter Geoghegan wrote:
> On Wed, Feb 26, 2014 at 2:10 PM, Andrew Dunstan <andrew(at)dunslane(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2014-02-27 01:09:07 | Re: jsonb and nested hstore |
Previous Message | Alvaro Herrera | 2014-02-27 01:01:54 | Re: jsonb and nested hstore |