Skip site navigation (1) Skip section navigation (2)

Re: Domain Support -- another round

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Rod Taylor <rbt(at)zort(dot)ca>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-patches(at)postgresql(dot)org
Subject: Re: Domain Support -- another round
Date: 2002-03-14 21:21:03
Message-ID: 200203142121.g2ELL3V09846@candle.pha.pa.us (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-patches
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.

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


Rod Taylor wrote:
> New set with most of Peters comments corrected.  Left the deal about
> schema though :)  Took nearly an hour to do a cvs diff for some reason
> this time (normally a couple of minutes is enough).
> 
> > Random nitpicking below.  Also, have you created a regression test?
> 
> They had been posted a few times and haven't changed.  (Attached
> anyway)
> 
> 
> > > +    <structfield>typnotnull</structfield> represents a NOT NULL
> > > +    constraint on a type.  Normally used only for domains.
> >
> > And unnormally...?
> 
> Unnormally is when someone sets it by hand on a type which isn't a
> domain -- I guess.  Corrected.
> 
> > > + <!entity createDomain       system "create_domain.sgml">
> >
> > I don't see this file included.
> 
> Other messages.  Full package included on this one however.
> 
> 
> 
> > > +  * MergeDomainAttributes
> > > +  *      Returns a new table schema with the constraints, types,
> and other
> > > +  *      attributes of the domain resolved for fields using the
> domain as
> > > +  * their type.
> >
> > I didn't know we had schemas yet.  You should probably not overload
> that
> > term to mean "a list of database objects".
> 
> Merge attributes says something very similar about inheritance and
> table schemas.  Kinda correct considering
> the variable used in both cases is *schema.
> 
> 
> The diff weirdness in regards to DROP DATABASE is probably because I
> started by copying the DROP DATABASE element, then altered it.  I
> don't know why it chose that method to do the diff though, but it is
> accurate.  Using -cd flags didn't make it any prettier.

[ Attachment, skipping... ]

[ Attachment, skipping... ]

[ Attachment, skipping... ]

[ Attachment, skipping... ]

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman(at)candle(dot)pha(dot)pa(dot)us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

In response to

pgsql-hackers by date

Next:From: Bruce MomjianDate: 2002-03-14 21:33:44
Subject: Re: Allowing usernames in pg_hba.conf
Previous:From: Bruce MomjianDate: 2002-03-14 21:20:18
Subject: Re: privileges regression problem on freebsd/alpha

pgsql-patches by date

Next:From: Bruce MomjianDate: 2002-03-14 21:44:22
Subject: Re: JDBC arrays
Previous:From: Bruce MomjianDate: 2002-03-14 21:20:46
Subject: Re: Docs on BOOTSTRAP relations

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group