launchpad-reviewers team mailing list archive

[Ian Booth <ian.booth@xxxxxxxxxxxxx>]

Ian Booth has proposed merging lp:~wallyworld/launchpad/accesspolicy-person-constraint into lp:launchpad.

Requested reviews:
  Robert Collins (lifeless)
  Stuart Bishop (stub)
Related bugs:
  Bug #1024990 in Launchpad itself: "private teams cannot access their +junk branches without a subscription"
  https://bugs.launchpad.net/launchpad/+bug/1024990

For more details, see:
https://code.launchpad.net/~wallyworld/launchpad/accesspolicy-person-constraint/+merge/122170

This database patch can be applied live. It's simply a unique constraint on "accesspolicy.person". The constraint needed to be added separately to the patch which created the column because a small model fix was needed first to allow person merges to proceed. This model change is in devel rev 15889 so this patch can be applied live after the next NDT.
-- 
https://code.launchpad.net/~wallyworld/launchpad/accesspolicy-person-constraint/+merge/122170
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

=== added file 'database/schema/patch-2209-30-2.sql'
--- database/schema/patch-2209-30-2.sql	1970-01-01 00:00:00 +0000
+++ database/schema/patch-2209-30-2.sql	2012-08-30 23:35:24 +0000
@@ -0,0 +1,7 @@
+-- Copyright 2012 Canonical Ltd.  This software is licensed under the
+-- GNU Affero General Public License version 3 (see the file LICENSE).
+SET client_min_messages=ERROR;
+
+CREATE UNIQUE INDEX accesspolicy__person__key ON accesspolicy USING btree (person) WHERE (person IS NOT NULL);
+
+INSERT INTO LaunchpadDatabaseRevision VALUES (2209, 30, 2);