New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for stacked profiler analysis #10551
Conversation
src/Factory/SessionFactory.php
Outdated
@@ -58,7 +58,7 @@ class SessionFactory | |||
*/ | |||
public function createSession(App\Mode $mode, App\BaseURL $baseURL, IConfig $config, Database $dba, ICache $cache, LoggerInterface $logger, Profiler $profiler, array $server = []) | |||
{ | |||
$stamp1 = microtime(true); | |||
$profiler->startRecording('parser'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this "parser" category? You replaced a few calls with "rendering" in other files of this same PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See some lines below. We use the "parser" key for the session stuff. And yes, this is quirky. And since "parser" is used for the cache handling, all the "parser" stuff moved to "rendering".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not call it "session" then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would be too easy.
Co-authored-by: Hypolite Petovan <hypolite@mrpetovan.com>
friendica/friendica-addons#1149 has to be merged at the same time. |
Until now the profiler could only be used for simple operations like database or file activities.
Now we support activities that call other actitivies that are profiled as well. With this we can better detect bottlenecks in more complex structures like the rendering.