[Rt-commit] rt branch, 4.2/init-once-in-db-insert-action, repushed

Alex Vandiver alexmv at bestpractical.com
Thu Mar 5 15:07:37 EST 2015


The branch 4.2/init-once-in-db-insert-action was deleted and repushed:
       was 8790845878430f90fb992e3156ca8eeb9afc79b3
       now ef0ff4bf9c23a013c737bac1d41c038bc4e7f249

1:  8790845 ! 1:  ef0ff4b Reduce the number of Init() calls per upgrade
    @@ -1,8 +1,16 @@
     Author: Alex Vandiver <alexmv at bestpractical.com>
     
    -    reduce the number of Init() calls per upgrade
    +    Reduce the number of Init() calls per upgrade
         
    -    it drops the time on an empty 3.8.17 database, upgraded to 4.2.10, from 6.5 minutes to 17s.
    +    Init() is a relatively heavy-weight operation, loading the configuration
    +    and all lib/ files, as well as performing post-load consistency checks.
    +    Running it once per insert operation occupies significant time, for no
    +    benefit.
    +    
    +    Only run RT::Init() during the first insert; no other action calls
    +    RT::Init(), so they are not elegible for the same change.  This drops
    +    the time to upgrade an empty 3.8.17 database to 4.2.10 from 6.5 minutes
    +    to 17s.
     
     diff --git a/sbin/rt-setup-database.in b/sbin/rt-setup-database.in
     --- a/sbin/rt-setup-database.in



More information about the rt-commit mailing list