Changes between Version 38 and Version 41 of Ticket #590


Ignore:
Timestamp:
10/09/13 14:04:59 (3 years ago)
Author:
jim
Comment:

I think we're creating a lot of URL aliases for things that don't need aliasing... I think therefore we can change the alias settings and the clean the unwanted items from the url_alias table, which is a heavily contended table.

This is Proposal O - updating description.

Long story short, we can nearly half the size of a very busy table that's hit on every request many times -- that's gotta be a good tweak!

Ed?

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #590

    • Property Add Hours to Ticket changed from 0.1 to 0.5
    • Property Total Hours changed from 14.65 to 15.75

  • Ticket #590 – Description

    v38 v41  
    1414'''J) Convert inline PHP into module code and features''' -- various parts (1-2 hours for block code; 4-6 hours to make Features be used across site), high reward, low risk -- {{{Eval()}}}uated code is much slower than PHP in files, especially since it can't be accelerated by APC or Zend Opcode cache... We have a few blocks and many views that are loaded from the database and evaluated. Ideally the blocks would be moved to the 'Transition Extras' module, and the views would be pushed into features. This work is good to do for maintainability and D7 upgrades, too. See: http://2bits.com/api/abuse-drupal-best-practices-your-own-peril-poor-performance.html and http://2bits.com/articles/free-your-content-php-moving-php-code-out-blocks-views-and-nodes.html 
    1515 
     16'''O) Stop making so many URL aliases for non-relevant pages, clean up url_alias table''' -- 1/2-1 hour, medium reward, only risk is that some links might break -- though given what is proposed that's unlikely. E.g. 
     17* 1,772 entries for: people/[name]/feed 
     18* 1,837 entries: user/[name]/contact 
     19* What's the point of people/[name] AND user/[name] being aliased when we always link to /people/*..? Taking out all /users/* aliases = 17,358 entries saved! 
    1620 
    1721=== To do: depending on spare time (JK to monitor, EM to confirm) ===