Thread: Re: [HACKERS] ALTER TABLE ... ADD COLUMN

Re: [HACKERS] ALTER TABLE ... ADD COLUMN

From
Bruce Momjian
Date:
Alvaro, I am seeing the attached regression failure, which I assume was
caused by your patch.  Is the new regression output correct, or is there
something wrong?

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

pgman wrote:
>
> Patch applied.  Thanks.
>
> ---------------------------------------------------------------------------
>
>
> Alvaro Herrera wrote:
> > En Fri, 4 Oct 2002 18:21:06 -0400
> > Alvaro Herrera <alvherre@atentus.com> escribi?:
> >
> > > Huh, I don't know where I got the idea you were (or someone else was?)
> > > in the position that attislocal should be reset.  I'll clean everything
> > > up and submit the patch I had originally made.
> >
> > All right, this is it.  This patch merely checks if child tables have
> > the column.  If atttypid and atttypmod are the same, the attributes'
> > attinhcount is incremented; else the operation is aborted.  If child
> > tables don't have the column, recursively add it.
> >
> > attislocal is not touched in any case.
> >
> > --
> > Alvaro Herrera (<alvherre[a]atentus.com>)
> > "El Maquinismo fue proscrito so pena de cosquilleo hasta la muerte"
> > (Ijon Tichy en Viajes, Stanislaw Lem)
>
> [ Attachment, skipping... ]
>
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 6: Have you searched our list archives?
> >
> > http://archives.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

--
  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
*** ./expected/misc.out    Fri Oct 18 22:34:51 2002
--- ./results/misc.out    Fri Oct 18 22:38:03 2002
***************
*** 377,436 ****
  --   SET a = text 'gazpacho'
  --   WHERE aa > 4;
  SELECT class, aa, a FROM a_star*;
!  class | aa | a
! -------+----+---
!  a     |  1 |
!  a     |  2 |
!  a     |    |
!  b     |  3 |
!  b     |  4 |
!  b     |    |
!  b     |    |
!  c     |  5 |
!  c     |  6 |
!  c     |    |
!  c     |    |
!  d     |  7 |
!  d     |  8 |
!  d     |  9 |
!  d     | 10 |
!  d     |    |
!  d     | 11 |
!  d     | 12 |
!  d     | 13 |
!  d     |    |
!  d     |    |
!  d     |    |
!  d     | 14 |
!  d     |    |
!  d     |    |
!  d     |    |
!  d     |    |
!  e     | 15 |
!  e     | 16 |
!  e     | 17 |
!  e     |    |
!  e     | 18 |
!  e     |    |
!  e     |    |
!  f     | 19 |
!  f     | 20 |
!  f     | 21 |
!  f     | 22 |
!  f     |    |
!  f     | 24 |
!  f     | 25 |
!  f     | 26 |
!  f     |    |
!  f     |    |
!  f     |    |
!  f     | 27 |
!  f     |    |
!  f     |    |
!  f     |    |
!  f     |    |
! (50 rows)
!
  --
  -- versions
  --
--- 377,383 ----
  --   SET a = text 'gazpacho'
  --   WHERE aa > 4;
  SELECT class, aa, a FROM a_star*;
! ERROR:  Relation "d_star" has no column "a"
  --
  -- versions
  --

======================================================================


Re: [HACKERS] ALTER TABLE ... ADD COLUMN

From
Alvaro Herrera
Date:
On Fri, Oct 18, 2002 at 10:43:30PM -0400, Bruce Momjian wrote:

> Alvaro, I am seeing the attached regression failure, which I assume was
> caused by your patch.  Is the new regression output correct, or is there
> something wrong?

Sorry for the mistake.  This one does pass the regression test.  A new
NOTICE to misc.output is in the patch.

--
Alvaro Herrera (<alvherre[a]dcc.uchile.cl>)
"El Maquinismo fue proscrito so pena de cosquilleo hasta la muerte"
(Ijon Tichy en Viajes, Stanislaw Lem)

Attachment

Re: [HACKERS] ALTER TABLE ... ADD COLUMN

From
Bruce Momjian
Date:
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.

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


Alvaro Herrera wrote:
> On Fri, Oct 18, 2002 at 10:43:30PM -0400, Bruce Momjian wrote:
>
> > Alvaro, I am seeing the attached regression failure, which I assume was
> > caused by your patch.  Is the new regression output correct, or is there
> > something wrong?
>
> Sorry for the mistake.  This one does pass the regression test.  A new
> NOTICE to misc.output is in the patch.
>
> --
> Alvaro Herrera (<alvherre[a]dcc.uchile.cl>)
> "El Maquinismo fue proscrito so pena de cosquilleo hasta la muerte"
> (Ijon Tichy en Viajes, Stanislaw Lem)

[ Attachment, skipping... ]

--
  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

Re: [HACKERS] ALTER TABLE ... ADD COLUMN

From
Bruce Momjian
Date:
Patch applied.  Thanks.

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


Alvaro Herrera wrote:
> On Fri, Oct 18, 2002 at 10:43:30PM -0400, Bruce Momjian wrote:
>
> > Alvaro, I am seeing the attached regression failure, which I assume was
> > caused by your patch.  Is the new regression output correct, or is there
> > something wrong?
>
> Sorry for the mistake.  This one does pass the regression test.  A new
> NOTICE to misc.output is in the patch.
>
> --
> Alvaro Herrera (<alvherre[a]dcc.uchile.cl>)
> "El Maquinismo fue proscrito so pena de cosquilleo hasta la muerte"
> (Ijon Tichy en Viajes, Stanislaw Lem)

[ Attachment, skipping... ]

--
  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