[hibernate-issues] [Hibernate-JIRA] Updated: (HHH-3532) schema update task should look for foreign key signature

schema update task should look for foreign key signature -------------------------------------------------------- Key: HHH-3532 URL: http://opensource.atlassian.com/projects/hibernate/browse/HHH-3532 Project: Hibernate Core Issue Type: Improvement Components: metamodel Affects Versions: 3.3.1 Reporter: Dan Allen Priority: Critical Attachments: HHH-3532.patch Original Estimate: 1 hour Remaining Estimate: 1 hour When the schema update task runs, it attempts to locate a foreign key in the database that matches the name provided in the Hibernate mapping information for that particular association. (The name of the foreign key is specified either in the foreign-key attribute in a mapping file or the @ForeignKey annotation). That much works fine. It's when a foreign key name is not provided, common in the case when standard JPA mappings are used, that problems arise. When a foreign key name is not provided, Hibernate generates a key name as follows: "FK" + hash code of referenced entity name + hash codes of columns The chances of an existing foreign key in the database using this generated name is slim to none (every DBA has their own conventions that they follow). Thus, as soon as the scheme update task hits a database under these conditions, duplicate foreign key definitions appear all over the place. If a foreign key name is not provided in the mapping, Hibernate should be looking instead to see if the signature of the foreign key matches that which is required to fulfill the mapping. In fact, I would even argue that the name is irrelevant and should only be used when actually *creating* the foreign key. Here is the existing logic in the generateSchemaUpdateScript in Configuration: ForeignKey fk = (ForeignKey) subIter.next(); boolean create = tableInfo == null || tableInfo.getForeignKeyMetadata( fk.getName() ) == null; It should be like this instead: ForeignKey fk = (ForeignKey) subIter.next(); boolean create = tableInfo == null || tableInfo.getForeignKeyMetadata( fk ) == null; getForeignKeyMetadata should retrieve based on the fk signature rather than the fk name.