Migrate DistributionSourcePackage database class to be backed by the database
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Triaged
|
High
|
Unassigned |
Bug Description
DistributionSou
used by the database class in the LP tree. DSP is a "virtual" thing.
However, once the fix for bug #43893 and/or bug #273495 (development
started under the banner of bug #43893 and the other bug was filed
later to cover those specific changes), a new database class will
exist - DistributionSou
table.
This has been done so that we can select and update the
bug_reporting_
outside of the module in which it's defined, and access to the
bug_reporting_
the non-database-backed DistributionSou
Phew. In short, it means that the interface to DSP has not changed.
The justification is that there wasn't time to do the complete job of
migrating DSP to being fully backed by the database. As you'll see, I
don't think this short-term measure will complicate the migration from
a virtual DSP class to a database-backed one greatly.
A plan was discussed during the pre-implementation phase of bug
#43893:
* Need to create new `DistributionSo
upload code at the same time as it creates new `SourcePackageName`
records. In addition to a bulk fix at roll-out time.
* Create a new `IDistributionS
create these things.
* Update the few places that create `DistributionSo
objects on the fly to use the utility.
* It'll be important to check if there are any nasty interactions
w.r.t. `DistributionSo
cron job, `cronscripts/
* Ideally we can ignore this cache for now, and everything will
work. However, it probably makes sense to fold everything in
`Distributio
BjornT commented on this at the time:
I'm not convinced we need a IDistributionSo
utility. We should get them through IDistribution, and there should
be one API to create both the DistributionSou
SourcePackageName at the same time.
tags: | added: tech-debt |
Changed in soyuz: | |
assignee: | Celso Providelo (cprov) → nobody |
Changed in soyuz: | |
assignee: | nobody → Edwin Grubbs (edwin-grubbs) |
Changed in launchpad: | |
assignee: | Edwin Grubbs (edwin-grubbs) → nobody |
tags: | added: target-picker |
It would be nice to have this table migrated for https:/ /dev.launchpad. net/Soyuz/ SourcePackageLi censes, so let's find a place in the 3.0 schedule for them.