I ran a set of crude benchmarking tests on my path_alias lookup patch. The results are reported here: http://drupal.org/node/75976 With the full test results attached here: http://drupal.org/files/issues/benchmark.txt While the number of database queries drops substantially, the actual processing time goes up, as both Requests per Second and Transfer Rate drop after the patch is applied. This may be due to the placement of variable_get() in the patch, but more likely, it seems that using the PHP-based check (which requires array handling) just shifts the load from the database to memory. I'd be interested to see other (more technically expert) people try this patch and test it to see if the results are the same. - Ken Rickard agentrickard On 7/29/06, Ken Rickard <agentrickard@gmail.com> wrote:
I have rerolled the path_alias lookup patch, this time with a working admin.
(I blame last night's failure on free OSCon beer!)
This patch allows admins who have enabled the path module to go to
admin/path/configure
And select paths to allow lookups for from a checkbox list.
Any paths not selected will not be invoked during drupal_lookup_path., These requests are intercepted within the url() function.
The issue is described here: http://drupal.org/node/75976
Again, this is a proof-of-concept patch, designed to see if it buys us any performance gains.
I do need some larger-scale testing, if anyone is willing.
- Ken Rickard agentrickard