Modify

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#916 closed enhancement (wontfix)

Improving performance

Reported by: alessio.spadaro@… Owned by: coderanger
Priority: normal Component: PrivateTicketsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description

The plugin, during the post-process phase, calls check_ticket_access for each ticket in the hdf. This is extremely slow as the number of the tickets grows.
Maybe a better solution could be:

  • build a list of id from the hdf
  • select all the ticket in a single call returning the relevant fields (id, reporter, owner and cc list)
  • use this list to filter out the hdf, removing tickets that doesn't match

I hope i'll be able to prepare a patch implementing this.

Attachments (0)

Change History (3)

comment:1 Changed 8 years ago by coderanger

This can't be done in 0.10 without breaking the encapsulation of the ticket API. If you need a faster database, try Postgres.

comment:2 Changed 8 years ago by alessio.spadaro@…

  • Resolution set to wontfix
  • Status changed from new to closed

I'm actually using Postgres, and with as little as a thousand ticket the load on the machine for, say, the active tickets standard report is quite high (for each query we have at least 1+n queries, where n is the outcome of the first query).

I thought it was possible obtaining such list with a single call on the query object, but double-checking it seems that this is not possible :(

I'll open a RFE on trac
(Btw, thanks for your work)

comment:3 Changed 8 years ago by coderanger

This is already possible with trunk, as they recently re-added the ability to use ids in queries. However I have no plans to port this plugin to trunk in the near future.

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.