Thread: cube patch
This is a comprehensive set of diffs (versus current CVS) that replaces those attached to the same message with the Earth Distance patches. Recent changes include changing the subscript in one place I forgot in the previous bugfix patch. A couple of added regression tests, which should help catch this mistake if it reappears. I also put in a limit of 100 dimensions in cube_large and cube_in to prevent making it easy to create very large cubes. Changing one define in cubedata.h will raise the limit if some needs more dimensions.
Attachment
Hey - can someone commit this patch to CVS so I can test it? I can't seem to get it to patch myself... Chris > -----Original Message----- > From: pgsql-patches-owner@postgresql.org > [mailto:pgsql-patches-owner@postgresql.org]On Behalf Of Bruno Wolff III > Sent: Tuesday, 10 September 2002 12:51 PM > To: pgsql-patches@postgresql.org > Subject: [PATCHES] cube patch > > > This is a comprehensive set of diffs (versus current CVS) that > replaces those > attached to the same message with the Earth Distance patches. > Recent changes include changing the subscript in one place I forgot > in the previous bugfix patch. A couple of added regression tests, which > should help catch this mistake if it reappears. > I also put in a limit of 100 dimensions in cube_large and cube_in to > prevent making it easy to create very large cubes. Changing one define > in cubedata.h will raise the limit if some needs more dimensions. >
Yes, I will get to it tomorrow. I delayed because Tom was away. --------------------------------------------------------------------------- Christopher Kings-Lynne wrote: > Hey - can someone commit this patch to CVS so I can test it? I can't seem > to get it to patch myself... > > Chris > > > -----Original Message----- > > From: pgsql-patches-owner@postgresql.org > > [mailto:pgsql-patches-owner@postgresql.org]On Behalf Of Bruno Wolff III > > Sent: Tuesday, 10 September 2002 12:51 PM > > To: pgsql-patches@postgresql.org > > Subject: [PATCHES] cube patch > > > > > > This is a comprehensive set of diffs (versus current CVS) that > > replaces those > > attached to the same message with the Earth Distance patches. > > Recent changes include changing the subscript in one place I forgot > > in the previous bugfix patch. A couple of added regression tests, which > > should help catch this mistake if it reappears. > > I also put in a limit of 100 dimensions in cube_large and cube_in to > > prevent making it easy to create very large cubes. Changing one define > > in cubedata.h will raise the limit if some needs more dimensions. > > > > > ---------------------------(end of broadcast)--------------------------- > TIP 3: if posting/reading through Usenet, please send an appropriate > subscribe-nomail command to majordomo@postgresql.org so that your > message can get through to the mailing list cleanly > -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073
Your patch has been added to the PostgreSQL unapplied patches list at:
http://candle.pha.pa.us/cgi-bin/pgpatches
I will try to apply it within the next 48 hours.
---------------------------------------------------------------------------
Bruno Wolff III wrote:
> This is a comprehensive set of diffs (versus current CVS) that replaces those
> attached to the same message with the Earth Distance patches.
> Recent changes include changing the subscript in one place I forgot
> in the previous bugfix patch. A couple of added regression tests, which
> should help catch this mistake if it reappears.
> I also put in a limit of 100 dimensions in cube_large and cube_in to
> prevent making it easy to create very large cubes. Changing one define
> in cubedata.h will raise the limit if some needs more dimensions.
[ Attachment, skipping... ]
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
Patch applied. Thanks. --------------------------------------------------------------------------- Bruno Wolff III wrote: > This is a comprehensive set of diffs (versus current CVS) that replaces those > attached to the same message with the Earth Distance patches. > Recent changes include changing the subscript in one place I forgot > in the previous bugfix patch. A couple of added regression tests, which > should help catch this mistake if it reappears. > I also put in a limit of 100 dimensions in cube_large and cube_in to > prevent making it easy to create very large cubes. Changing one define > in cubedata.h will raise the limit if some needs more dimensions. [ Attachment, skipping... ] > > ---------------------------(end of broadcast)--------------------------- > TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073