Review: Approve db
BinaryPackagePath.path should be UNIQUE NOT NULL.

Shouldn't this table be called BinaryPackageReleasePath to match 
BinaryPackageReleaseContents?

Both columns in BinaryPackageReleaseContents should be NOT NULL.

patch-2208-76-0.sql

You likely will want this construct to ensure a path exists in the 
BinaryPackageReleasePaths table. This will avoid race conditions in scripts 
(not running in serializable isolation):

insert into binarypackagepaths (path) select 'foo' where not exists (select * 
from binarypackagepaths where path='foo');
-- 
https://code.launchpad.net/~stevenk/launchpad/db-add-bprc/+merge/64783
Your team Launchpad code reviewers is requested to review the proposed merge of 
lp:~stevenk/launchpad/db-add-bprc into lp:launchpad/db-devel.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to