Re: hstore parser incorrectly handles malformed input - Mailing list pgsql-bugs

From Robert Haas
Subject Re: hstore parser incorrectly handles malformed input
Date
Msg-id CA+TgmoYzuJ6noRk-amXVW1vXqVErWRf6_JzpunhgbSQ6nTz5bA@mail.gmail.com
Whole thread Raw
In response to Re: hstore parser incorrectly handles malformed input  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-bugs
On Thu, Apr 26, 2012 at 9:12 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> I wrote:
>> Ryan Kelly <rpkelly22@gmail.com> writes:
>>> In my mind, all of these should have been rejected as erroneous input.
>>> To that end, I have attached a patch which causes all of these inputs
>>> to be rejected as invalid.
>
>> Hm, I would have expected all three of these to result in "a" having
>> an empty-string value. =A0I see nothing in the hstore documentation
>> suggesting that I must write a=3D>"" or some such to get an empty value,
>
> Attached is an alternative patch that fixes it that way.
>
> Does anybody else have an opinion as to which of these solutions is
> more preferable? =A0And should we regard this as a back-patchable bug
> fix, or a definition change suitable only for HEAD?

I vote for not back-patching, regardless of exactly what we decide to do he=
re.

--=20
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgsql-bugs by date:

Previous
From: Robert Haas
Date:
Subject: Re: BUG #6624: Tab completion of identifier containing single backslash triggers warnings
Next
From: Ryan Kelly
Date:
Subject: Re: BUG #6629: Creating a gist index fails with "too many LWLocks taken"