Refactor SQL query time calculation to use Django aggregation

jazzband / django-silk

Silky smooth profiling for Django

MIT License

4.4k stars 334 forks source link

Closed beltagymohamed closed 1 month ago

beltagymohamed commented 2 months ago

Refactored time_spent_on_sql_queries property in the Request model:
- Replaced manual summation of time_taken with a Django aggregate query.
Updated relevant test case to reflect the changes.

This improves the efficiency of the code.

beltagymohamed commented 1 month ago

@albertyw the PR is ready, sorry for the big number of commits and workflow