On Thu, Jun 17, 2010 at 4:50 PM, Josh Berkus <j...@agliodbs.com> wrote: > On 6/17/10 1:40 PM, Robert Haas wrote: >> On Thu, Jun 17, 2010 at 4:39 PM, Josh Berkus <j...@agliodbs.com> wrote: >>>> Since there are no other votes for that option (or, indeed, any other >>>> option), I'm going to go with my original instinct and change hstore >>>> => text[] to hstore & text[]. Patch to do that is attached. >>> If what that operator is doing is appending an array of text to an >>> Hstore, shouldn't we use || instead? >> >> It isn't. || already does what you're saying. > > So what *does* it do? > > OK, so after a brief poll on IRC, one reason you're not getting coherent > feedback on this is that few people understand the operators which > hstore 9.0 already uses and which are new for 9.0, let alone what new > operators are proposed for each thing. I know I've completely lost > track, particularly since doc patches haven't kept up with the code > changes. I've reread most of this thread and it doesn't help me. > > On the other hand, maybe less feedback is less bikeshedding. You decide.
Well, they are documented, so you can read up on them... http://developer.postgresql.org/pgdocs/postgres/hstore.html This isn't a critical issue in desperate need of community input; we just need to resolve it one way or the other so we can move on to the next thing. I'm still inclined to go ahead and apply the patch I attached upthread, because that is less work for me than doing anything else... -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise Postgres Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
