Re: Update hstore % Doc
| От | David E. Wheeler |
|---|---|
| Тема | Re: Update hstore % Doc |
| Дата | |
| Msg-id | AF65AC3F-0F48-463F-A57A-F7C23F5EEAB4@pgexperts.com обсуждение исходный текст |
| Ответ на | Re: Update hstore % Doc (Tom Lane <tgl@sss.pgh.pa.us>) |
| Ответы |
Re: Update hstore % Doc
Re: Update hstore % Doc |
| Список | pgsql-hackers |
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 looklike 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 % afterall the arguments, but coming back to it fresh it just looks bizarre to me. Maybe that ship has sailed, though, andI'm just being difficult. Best, David
В списке pgsql-hackers по дате отправления: