[Rt-devel] PATCH - History display & performance (ShowTransaction)
sturner at MIT.EDU
Tue Nov 29 10:07:52 EST 2005
At Monday 11/28/2005 06:27 PM, Jesse Vincent wrote:
>On Mon, Nov 28, 2005 at 05:35:15PM -0500, Stephen Turner wrote:
> > Here's one more suggestion for speeding up the display of transactions. I
> > noticed that in Ticket/Elements/ShowTransaction, the link for
> displaying an
> > Email Record is formed by using $Transaction->Attachments->First to get
> > attachment ID. This repeats the query to get the transaction attachments
> > for each transaction. But all the ticket's attachments have already been
> > cached up front for performance reasons, so this additional query isn't
> > necessary.
> > I've attached a patch that gets the attachment ID from the cached list
> > instead of re-reading it from the database. RT 3.4.2.
> > Please let me know if I've misunderstood this!
>Hm. Does that actually change what database queries are done?
Yes - with the standard ShowTransaction component I'm seeing this query
executed for each "email record" transaction:
Attachments main WHERE ((main.TransactionId = '112812')) ORDER BY main.id
Using the cached attachment collection eliminates the queries and seems to
get the same result.
More information about the Rt-devel