Обсуждение: pgsql: Remove hstore's text => text operator.
Remove hstore's text => text operator. Since PostgreSQL 9.0, we've emitted a warning message when an operator named => is created, because the SQL standard now reserves that token for another use. But we've also shipped such an operator with hstore. Use of the function hstore(text, text) has been recommended in preference to =>(text, text). Per discussion, it's now time to take the next step and stop shipping the operator. This will allow us to prohibit the use of => as an operator name in a future release if and when we wish to support the SQL standard use of this token. The release notes should mention this incompatibility. Patch by me, reviewed by David Wheeler, Dimitri Fontaine and Tom Lane. Branch ------ master Details ------- http://git.postgresql.org/pg/commitdiff/b60653bc0b75b7f3b5dda0a2968a22129aafb2b2 Modified Files -------------- contrib/hstore/Makefile | 3 +- contrib/hstore/hstore--1.0--1.1.sql | 7 + contrib/hstore/hstore--1.1.sql | 524 +++++++++++++++++++++++++++++++++++ contrib/hstore/hstore.control | 2 +- doc/src/sgml/hstore.sgml | 17 +- 5 files changed, 535 insertions(+), 18 deletions(-)
On 8 November 2011 02:51, Robert Haas <rhaas@postgresql.org> wrote: > Remove hstore's text => text operator. > > Since PostgreSQL 9.0, we've emitted a warning message when an operator > named => is created, because the SQL standard now reserves that token > for another use. But we've also shipped such an operator with hstore. > Use of the function hstore(text, text) has been recommended in > preference to =>(text, text). Per discussion, it's now time to take > the next step and stop shipping the operator. This will allow us to > prohibit the use of => as an operator name in a future release if and > when we wish to support the SQL standard use of this token. > > The release notes should mention this incompatibility. > > Patch by me, reviewed by David Wheeler, Dimitri Fontaine and Tom Lane. There's a typo here: +/* contrib/hstore/hstore-1.0-1.1.sql */ Should be: +/* contrib/hstore/hstore-1.0--1.1.sql */ -- Thom Brown Twitter: @darkixion IRC (freenode): dark_ixion Registered Linux user: #516935 EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
On 11/07/2011 09:51 PM, Robert Haas wrote: > Remove hstore's text => text operator. > > This is still open on the commitfest ... cheers andrew