Re: contrib/ intarray, ltree, intagg broken(?) by array changes

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Oleg Bartunov <oleg(at)sai(dot)msu(dot)su>, Teodor Sigaev <teodor(at)stack(dot)net>, mlw <markw(at)mohawksoft(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: contrib/ intarray, ltree, intagg broken(?) by array changes
Date: 2002-09-02 02:51:09
Message-ID: 200209020251.g822p9j21659@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Can someone address the intagg issue here, or is the code OK?

---------------------------------------------------------------------------

Tom Lane wrote:
> Joe Conway and I have just committed some changes in the internal
> representation of Postgres arrays: an element-type-OID field is added to
> the array header, and alignment calculations are now done the same way
> as in ordinary tuple storage, instead of taking shortcuts. I believe
> that these changes need to be reflected into the intarray, ltree, and
> intagg contrib modules.
>
> intarray and ltree both seem to be mapping their own declarations onto
> arrays using largely-similar code. But while intarray fails its
> regression test, I find ltree still passes. So I'm confused about what
> that code is really doing and don't want to touch it.
>
> I tried to fix intagg, but since there is no regression test for it
> I'm unsure whether it's okay.
>
> Could you folks take a look at CVS tip and see what changes are needed,
> if any?
>
> In the longer run, it might be possible to improve these routines to be
> array-type-polymorphic using the new features. But with the 7.3 beta
> date nearly upon us, I'd counsel first making the existing functionality
> work again...
>
> regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo(at)postgresql(dot)org so that your
> message can get through to the mailing list cleanly
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2002-09-02 02:51:52 Re: Think I see a btree vacuuming bug
Previous Message Bruce Momjian 2002-09-02 02:34:55 Re: CVS broken - large file support?