launchpad-dev team mailing list archive

[Julian Edwards <julian.edwards@xxxxxxxxxxxxx>]

On Tuesday 22 February 2011 22:50:11 Robert Collins wrote:
> Todays note about performance is a note about some storm/sqlobject glue.
> 
> Code like this:
> ...
> if bugtask.distribution is None:
>     ...
> 
> is probably wrong: it should be
> 
> ...
> if bugtask.distributionID is None:
>     ...

I did read below but I'm not sure of the significance (if any) of this being 
an SQLObject-style model class.

Does the same thing happen with Storm-style ones, and we'd need to reference 
bugtask.distribution_id ?

While I'm thinking about it, why do we have this inconsistency between 
field_id and fieldID for Storm vs SQLObject?

And finally, DEATH to SQLObject classes.