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-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 intended to

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

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 syber@gmail.com 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

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 :

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. /mst Seriously, you obviously think this is worth spending time on, so spend the time on it, and it will be fixed for

[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