I ran a set of crude benchmarking tests on my path_alias lookup patch. The results are reported here:<br><br><a href="http://drupal.org/node/75976">http://drupal.org/node/75976</a><br><br>With the full test results attached here:
<br><br><a href="http://drupal.org/files/issues/benchmark.txt">http://drupal.org/files/issues/benchmark.txt</a><br><br>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.
<br><br>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.<br><br>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.
<br><br>- Ken Rickard<br>agentrickard<br><br><br><div><span class="gmail_quote">On 7/29/06, <b class="gmail_sendername">Ken Rickard</b> <<a href="mailto:agentrickard@gmail.com">agentrickard@gmail.com</a>> wrote:</span>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div>I have rerolled the path_alias lookup patch, this time with a working admin.<br><br>(I blame last night's failure on free OSCon beer!)
<br><br>This patch allows admins who have enabled the path module to go to<br><br>admin/path/configure
<br><br>And select paths to allow lookups for from a checkbox list.<br><br>Any paths not selected will not be invoked during drupal_lookup_path., These requests are intercepted within the url() function.<br><br>The issue is described here:
<a href="http://drupal.org/node/75976" target="_self" onclick="return top.js.OpenExtLink(window,event,this)">http://drupal.org/node/75976</a> <br><br>Again, this is a proof-of-concept patch, designed to see if it buys us any performance gains.
<br><br>I do need some larger-scale testing, if anyone is willing.
<br></div><div><span class="sg"><br>- Ken Rickard<br>agentrickard<br>
</span></div></blockquote></div><br>