Comment 23 for bug 1904737

Revision history for this message
Jason Stephenson (jstephenson) wrote :

I have new WIP branch that addresses all of Galen's concerns in comment #19.

I have also worked out a solution to the query performance. By adding a new index on asset.copy.circ_lib where deleted is false, the performance of the new queries is actually improved over the queries using the original index. This new index also improves the performance of the original queries when they are modified to use it.

I have also attached a revised query bundle that includes a modified version of the new query to exercise the new index on asset.copy so that others can compare the performance. The README in the bundle has also been modified to include the new index definition.

I have more testing and possible tweaking to do, but I expect to have a pull request branch ready in a week or so. I will rename the branch when I share it.