Re: [Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-23 Thread David Wright
Jonathan Rockway wrote: * On Mon, Feb 23 2009, Jason Gottshall wrote: Oleg Pronin wrote: I use many actions that take params from config in runtime, for example sub pay_for_vip : Private { ... my $price = $c->cfg->{vip}{price}; } As I understand it, this is N

Re: [Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-23 Thread Jonathan Rockway
* On Mon, Feb 23 2009, Jason Gottshall wrote: > Oleg Pronin wrote: >> I use many actions that take params from config in runtime, for example >> sub pay_for_vip : Private { >>... >>my $price = $c->cfg->{vip}{price}; >> } > > As I understand it, this is NOT the way config is in

Re: [Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-23 Thread Jason Gottshall
Oleg Pronin wrote: I use many actions that take params from config in runtime, for example sub pay_for_vip : Private { ... my $price = $c->cfg->{vip}{price}; } As I understand it, this is NOT the way config is intended to work. All the config for your component (controller

Re: [Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-22 Thread Jonathan Rockway
* On Sun, Feb 22 2009, Oleg Pronin wrote: > And in my deepest opinion we should now spend CPU time anywhere even if it is > not a > concern. You forgot to attach the patch. Seriously, you obviously think this is worth spending time on, so spend the time on it, and it will be fixed for everyone

Re: [Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-22 Thread Oleg Pronin
1% here and 1% there. You won't detect any place that takes longer. All of load is 1%+1%+ Don't you agree ? Plus a number of plugins calls ->config on startup which slow downs startup as well) P.S. I use many actions that take params from config in runtime, for example sub pay_for_vip : Pr

Re: [Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-22 Thread Oleg Pronin
*we should now -> we should not :-) 2009/2/22 Oleg Pronin > 1% here and 1% there. You won't detect any place that takes longer. All of > load is 1%+1%+ > Don't you agree ? Plus a number of plugins calls ->config on startup which > slow downs startup as well) > > P.S. > I use many actions t

Re: [Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-21 Thread Jonathan Rockway
* On Sat, Feb 21 2009, Oleg Pronin wrote: > I use Catalyst in extremely loaded projects (currently 60.000.000 pageloads / > day). > Thereforce i'm perfomance paranoid man. > > One of 'black stones' is the ->config method. It has dramatically slow > perfomance at > config_method: 1 wallclock secs

[Catalyst] Maybe there is a need for some speedups of 'config' method ?

2009-02-21 Thread Oleg Pronin
Hello. Catalyst is the best MVC ever. Thanks for doing your work guys! I use Catalyst in extremely loaded projects (currently 60.000.000 pageloads / day). Thereforce i'm perfomance paranoid man. One of 'black stones' is the ->config method. It has dramatically slow perfomance at config_method: 1