openstack
/
keystone
Code Issues Proposed changes

Redundant unique constraint 

federation extension have redundant unique constraint. The unique
constraint is totally useless, as (id, idp_id) columns are
columns of a composite primary key, which is unique by definition.

Change-Id: I93648ab89b26d7e77875a1faa8575a146224232f
Partial-Bug: #1292591
This commit is contained in:
Ilya Pekelny 2014-04-08 15:20:51 +03:00
parent 326f57c271
commit ca370e2ac5
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
keystone/contrib/federation/backends/sql.py
View File

@ -27,7 +27,6 @@ class FederationProtocolModel(sql.ModelBase, sql.DictBase):
idp_id = sql.Column(sql.String(64), sql.ForeignKey('identity_provider.id',
ondelete='CASCADE'), primary_key=True)
mapping_id = sql.Column(sql.String(64), nullable=False)
__table_args__ = (sql.UniqueConstraint('id', 'idp_id'), dict())
@classmethod
def from_dict(cls, dictionary):