jazzband / django-silk

Silky smooth profiling for Django
MIT License
4.45k stars 335 forks source link

Refactor SQL query time calculation to use Django aggregation #720

Closed beltagymohamed closed 2 months ago

beltagymohamed commented 3 months ago

This will fix a "FIXME" in the tests, and will make the code faster by utilizing db aggregation instead of python sum

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.82%. Comparing base (1cb4623) to head (96390db). Report is 9 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #720 +/- ## ========================================== + Coverage 86.76% 86.82% +0.05% ========================================== Files 52 52 Lines 2115 2117 +2 ========================================== + Hits 1835 1838 +3 + Misses 280 279 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.