Re: GIST and TOAST

"Teodor Sigaev" <teodor@sigaev.ru> writes:

>> It doesn't actually crash, it just fails CHECKARRVALID. I added an assertion
>> in there to cause it to generate a core dump.
>
> Wow, catch that, see attached patch.
>
> g_int_decompress doesn't returns detoasted array in case it was empty.
> Previously it was safe because empty array never has been toasted.

Ah, thanks a bunch.

> Should I commit it or you'll include in your patch?

I'll include it in the patch I guess since it's fine the way it is until the
patch hits.

Now I'll try running the regressions again with the gist datatypes like hstore
etc all packed as well.


--  Gregory Stark EnterpriseDB          http://www.enterprisedb.com