maykinmedia / commonground-api-common

Shared functions between Common Ground API components
0 stars 0 forks source link

Performance improvements #3

Closed sergei-maertens closed 1 year ago

sergei-maertens commented 1 year ago

While investigating open-zaak/open-zaak#1271, it turns out a lot of (create endpoint) queries can be avoided or optimized in a number of ways.

Required for https://github.com/open-zaak/open-zaak/pull/1299

codecov-commenter commented 1 year ago

Codecov Report

Merging #3 (0c476db) into main (e057775) will decrease coverage by 0.17%. The diff coverage is 45.00%.

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
- Coverage   53.16%   52.99%   -0.18%     
==========================================
  Files          78       78              
  Lines        3566     3604      +38     
  Branches      521      530       +9     
==========================================
+ Hits         1896     1910      +14     
- Misses       1573     1595      +22     
- Partials       97       99       +2     
Impacted Files Coverage Δ
vng_api_common/serializers.py 60.50% <0.00%> (-1.43%) :arrow_down:
vng_api_common/utils.py 58.27% <36.36%> (-1.89%) :arrow_down:
vng_api_common/views.py 37.60% <40.00%> (+1.01%) :arrow_up:
vng_api_common/audittrails/viewsets.py 38.83% <50.00%> (+0.29%) :arrow_up:
vng_api_common/caching/etags.py 79.56% <57.14%> (-8.90%) :arrow_down:
vng_api_common/conf/api.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more