stadust / pointercrate

Rewrite of the https://pointercrate.com backend in Rust
MIT License
80 stars 52 forks source link

Cache stats viewer scores #129

Closed stadust closed 4 months ago

stadust commented 5 months ago

Cache all player scores in a new column of the players table, to improve performance of stats viewer.

License Acceptance

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 61.30952% with 65 lines in your changes are missing coverage. Please review.

Project coverage is 26.92%. Comparing base (c36253c) to head (4277a31). Report is 1 commits behind head on master.

Files Patch % Lines
pointercrate-demonlist-api/src/pages.rs 3.44% 28 Missing :warning:
pointercrate-demonlist/src/player/paginate.rs 7.69% 12 Missing :warning:
pointercrate-demonlist-api/src/endpoints/player.rs 0.00% 11 Missing :warning:
pointercrate-demonlist/src/demon/patch.rs 0.00% 5 Missing :warning:
pointercrate-demonlist/src/player/patch.rs 89.65% 3 Missing :warning:
pointercrate-demonlist/src/record/patch.rs 33.33% 2 Missing :warning:
pointercrate-demonlist/src/nationality/get.rs 94.73% 1 Missing :warning:
pointercrate-demonlist/src/nationality/mod.rs 95.00% 1 Missing :warning:
pointercrate-demonlist/src/record/mod.rs 0.00% 1 Missing :warning:
pointercrate-demonlist/src/record/post.rs 66.66% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #129 +/- ## ========================================== + Coverage 24.24% 26.92% +2.68% ========================================== Files 113 112 -1 Lines 8436 8593 +157 ========================================== + Hits 2045 2314 +269 + Misses 6391 6279 -112 ```

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