Skip to content
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

[IMP] http,server: add query count and request times in logs #25022

Closed

Conversation

d-fence
Copy link
Contributor

@d-fence d-fence commented Jun 1, 2018

When there is a performance issue, it's sometimes difficult to discover
which request increased the query count or its duration.

With this commit, the query count, the system time, the user time and
the real time are displayed in the logs at the end of each werkzeug
request line.

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@d-fence d-fence force-pushed the master-imp-log-query-count-times-moc branch from 92bc2ff to 24f90d8 Compare June 4, 2018 14:07
@C3POdoo C3POdoo added the RD research & development, internal work label Jun 7, 2018
d-fence and others added 11 commits June 25, 2018 10:03
When there is a performance issue, it's sometimes difficult to discover
which request increased the query count or its duration.

With this commit, the query count, the system time, the user time and
the real time are displayed in the logs at the end of each werkzeug
request line.
query_count/sys time/user time/elapsed time
@d-fence d-fence force-pushed the master-imp-log-query-count-times-moc branch from 99240c9 to 56a2a33 Compare June 25, 2018 08:40
@d-fence
Copy link
Contributor Author

d-fence commented Dec 11, 2018

Done here c44fe91

@d-fence d-fence closed this Dec 11, 2018
@d-fence d-fence deleted the master-imp-log-query-count-times-moc branch December 11, 2018 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants