Launchpad itself

Bug.indexed_messages unused?

Bug #606911 reported by Jeroen T. Vermeulen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Invalid
High
Unassigned

Bug Description

I can't find any code using Bug.indexed_messages. It fires off queries that we're having some performance problems with at the moment, so if it's not needed then this would be a good time to get rid of it.

Tags: lp-bugs
Revision history for this message
Graham Binns (gmb) wrote :

It turns out that it's not unused; it's exported as 'messages' for API use, so taking it away would be a Bad Thing.

Changed in malone:
status: New → Invalid
Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 606911] Re: Bug.indexed_messages unused?

Is it used by API clients though? Or just exported...

Revision history for this message
Deryck Hodge (deryck) wrote :

If we can't determine if it is used by API clients, we could change it to a method, i.e. getIndexedMessages, which would give API clients access to this data if they need it but save some queries without this being a property.

Changed in malone:
status: Invalid → Incomplete
status: Incomplete → Triaged
importance: Undecided → High
Revision history for this message
Robert Collins (lifeless) wrote :

So its used, and has been optimised, with model changes.

Changed in launchpad:
status: Triaged → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.