On Aug 6, 2010, at 3:16 PM, Tom Lane wrote: >> I noticed that the hstore docs still document the => operator instead >> of %. This patch changes that. > > It looks to me like you are changing the examples of the I/O > representation ... which did NOT change.
Hrm? The first few examples at the top? I find them confusing because there are no single quotes around them, so they look like the use of the deprecated => operator (especially the first two). Just look at: http://developer.postgresql.org/pgdocs/postgres/hstore.html Maybe that's standard in the docs, but it seems weird to me. It's a minor point, though. We definitely need to document the `text % text` constructor rather than the deprecated `text => text` constructor. And I hate to say it, but % is awful. Sorry, I know I'm probably opening a can of worms here, and I did finally push % after all the arguments, but coming back to it fresh it just looks bizarre to me. Maybe that ship has sailed, though, and I'm just being difficult. Best, David -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers