Menu

#109 Creating preferences.php is a waste of time

closed-accepted
nobody
None
5
2004-06-04
2003-12-09
Lee Essen
No

The get_preference() function is called a number of
times in tiki-setup before it get's to the point where
it creates the preferences.php "cache" file.

However, the first time get_preference() queries the db
and caches (in memory) the whole preferences table and
uses it for future lookups, that means that after the
first get_preference it's all in memory anyway.

So ... what is the point of creating the
preferences.php file? ... it doesn't stop any database
queries, add's the overhead of creating and reading it,
and breaks things in the cluster world.

This explains why performance is better when you take
that bit of code out!

So ... much better patch attached, just removal of code!

Discussion

  • Lee Essen

    Lee Essen - 2003-12-09

    gzipped tar file containing two patches

     
  • teedog

    teedog - 2004-06-04
    • status: open --> closed-accepted
     
  • teedog

    teedog - 2004-06-04

    Logged In: YES
    user_id=807810

    According to
    http://sourceforge.net/mailarchive/message.php?msg_id=6681187
    this patch was committed? I looked at tiki-setup.php and
    the patch is indeed in. Closing this now. Thanks Lee.

     

Log in to post a comment.