search

appsembler / figures

Reporting and data retrieval app for Open edX
MIT License
44 stars 37 forks source link

Improve performance for site-monthly-metrics/active_users/ #205

Closed johnbaldwin closed 4 years ago

johnbaldwin commented 4 years ago

Fix site monthly metrics 'active_user' endpoint by storing a versioned record of prior month's data

Details here: https://appsembler.atlassian.net/wiki/spaces/RT/pages/343703912/Figures+API+performance#Solution-for-Site-Monthly-Metrics-%E2%80%9CActive-User%E2%80%9D-Request-Timeout

codecov-io commented 4 years ago

Codecov Report

Merging #205 into master will increase coverage by 0.06%. The diff coverage is 93.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   92.22%   92.29%   +0.06%     
==========================================
  Files          32       35       +3     
  Lines        1736     1829      +93     
==========================================
+ Hits         1601     1688      +87     
- Misses        135      141       +6
Impacted Files Coverage Δ
...es/management/commands/backfill_figures_metrics.py 85.71% <85.71%> (ø)
figures/models.py 96.98% <90.90%> (-0.93%) :arrow_down:
figures/backfill.py 95.65% <95.65%> (ø)
figures/admin.py 100.00% <100.00%> (ø)
figures/metrics.py 87.98% <100.00%> (+1.06%) :arrow_up:
figures/migrations/0010_site_monthly_metrics.py 100.00% <100.00%> (ø)
figures/views.py 91.74% <100.00%> (-0.06%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7afeced...8cb2ecd. Read the comment docs.