| From: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> |
|---|---|
| To: | pgsql-bugs(at)postgresql(dot)org |
| Subject: | small hstore bugfixes for 9.0 (w/patch) |
| Date: | 2010-09-15 21:25:39 |
| Message-ID: | 87bp7yiu4y.fsf@news-spur.riddles.org.uk |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | 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 | Content-Type | Size |
|---|---|---|
| hs.patch | text/x-patch | 1.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2010-09-16 01:30:36 | Re: small hstore bugfixes for 9.0 (w/patch) |
| Previous Message | Tom Lane | 2010-09-15 19:15:59 | Re: BUG #5658: 'at time zone {timezone description}' can't recognize many time zone abbreviations. |