how about adding a directive in db.props (much like the
indexInsideTableDirective) ?
The problem with the code generation for postgres is that you
can't specify on DELETE or on UPDATE with the syntax given:
ADD CONSTRAINT $fk.LocalColumnNames FOREIGN KEY ($fk.LocalColumnNames)
REFERENCES $fk.ForeignTableName ($fk.ForeignColumnNames);
The alternative is then to produce contraint triggers whose syntax
is dependant on which version of postgres you are using.
The dtd changes I made and the changes to ForeignKey.java are
completely independent of the circular reference problem.
I can go ahead and add the correct syntax for contraint triggers
but it will only work for postgres 7.1.x.
let me know,
mike
On Thu, Jun 21, 2001 at 02:03:30PM -0700, John McNally wrote:
> The reason the fk's were moved to the end was prevent problems with
> circular references and ordering issues in general. What happens if
> table(id) is defined after the proposed line?
>
> john mcnally
>
> Mike Haberman wrote:
> >
> > Got all the code done and it's tested. Waiting for approval :)
> >
> > mike
> >
> > On Thu, Jun 21, 2001 at 10:36:24AM -0500, Mike Haberman wrote:
> > > I would like to add the onDelete and onUpdate attributes to
> > > the foreign key in the dtd.
> > >
> > > <!ELEMENT foreign-key (reference+)>
> > > <!ATTLIST foreign-key
> > > foreignTable CDATA #REQUIRED
> > > onUpdate (cascade|set null|restrict|none) "none"
> > > onDelete (cascade|set null|restrict|none) "none"
> > > >
> > >
> > >
> > > Also, at least for postgres, I would like to generate the foreign key
> > > stuff in line (instead of generating it after the table). It's much
> > > more elegant (and easier) to generate :
> > >
> > > colname type REFERENCES table(id) ON DELETE cascade,
> > >
> > > than
> > >
> > > CREATE TRIGGER if_dist_exists
> > > BEFORE INSERT OR UPDATE ON films FOR EACH ROW
> > > EXECUTE PROCEDURE check_primary_key ('col', 'table', 'col');
> > >
> > > CREATE TRIGGER if_film_exists
> > > BEFORE DELETE OR UPDATE ON distributors FOR EACH ROW
> > > EXECUTE PROCEDURE check_foreign_key (1, 'CASCADE', 'col', 'table', 'col');
> > >
> > >
> > > mike
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> > --
> > -------------------------------------------------
> > I am Vinz, Vinz Clortho. Keymaster of Gozer,
> > Volguus Zildrohar, Lord of the Sebouillia.
> > Are you the Gatekeeper?
> > -------------------------------------------------
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
--
-------------------------------------------------
I am Vinz, Vinz Clortho. Keymaster of Gozer,
Volguus Zildrohar, Lord of the Sebouillia.
Are you the Gatekeeper?
-------------------------------------------------
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
