small hstore bugfixes for 9.0 (w/patch) - Mailing list pgsql-bugs

From Andrew Gierth
Subject small hstore bugfixes for 9.0 (w/patch)
Date
Msg-id 87bp7yiu4y.fsf@news-spur.riddles.org.uk
Whole thread Raw
Responses Re: small hstore bugfixes for 9.0 (w/patch)  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: small hstore bugfixes for 9.0 (w/patch)  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-bugs
Two small fixes for hstore-new.

The hstore_compat one is arguable as to what is the best approach; the
assert that was there was just wrong, but I have been unable after
considerable searching to find any architectures that would fail the
check. The version in this patch will just treat any old-format
non-empty hstore as being invalid on a platform where the upgrade to
the new format would fail. (And this version _is_ tested on at least
i386 and amd64, where upgrading works.)

The gist one is just that the old code was abusing DatumGetHStoreP by
applying it to something that wasn't an hstore. This didn't matter
before the format upgrade code was put in, and it didn't show up in
tests because you need to index a very large number of hstores before
any problem shows up.

--
Andrew (irc:RhodiumToad)


Attachment

pgsql-bugs by date:

Previous
From: Tom Lane
Date:
Subject: Re: BUG #5658: 'at time zone {timezone description}' can't recognize many time zone abbreviations.
Next
From: Tom Lane
Date:
Subject: Re: small hstore bugfixes for 9.0 (w/patch)