FR, SQL backend and Foreign keys

[Alan DeKok]

Fabien COMBERNOUS wrote:
> here a patch proposed :

  A bit of explanation would help.

  It looks reasonable, but I'd want someone to try it before putting it
into the server.

> --- schema_orig.sql    2011-09-28 10:42:08.000000000 +0200

  *PLEASE* use full paths.  There are 4-5 SQL backends in the server.
Which one is this modifying?  Don't make us guess...


> +++ schema_withFK.sql    2011-09-28 10:46:42.000000000 +0200
> @@ -63,8 +63,8 @@
>    op char(2) NOT NULL DEFAULT '==',
>    value varchar(253) NOT NULL default '',
>    PRIMARY KEY  (id),
> -  KEY username (username(32))
> -) ;
> +  KEY username (username)
> +) ENGINE=InnoDB;
> 
>  #
>  # Table structure for table 'radgroupcheck'
> @@ -117,8 +117,10 @@
>    username varchar(64) NOT NULL default '',
>    groupname varchar(64) NOT NULL default '',
>    priority int(11) NOT NULL default '1',
> -  KEY username (username(32))
> -) ;
> +  INDEX username (username),
> +  FOREIGN KEY (`username`) REFERENCES radcheck(username)
> +    ON DELETE CASCADE ON UPDATE CASCADE
> +) ENGINE=InnoDB;
>