boxine / django-huey-monitor

Django based tool for monitoring huey task queue: https://github.com/coleifer/huey
GNU General Public License v3.0
86 stars 20 forks source link

Fix #46 By increment existing TaskProgressModel instances #68

Closed jedie closed 2 years ago

jedie commented 2 years ago

Don't create a new TaskProgressModel instances for every ProcessInfo.update() call and increment a existing TaskProgressModel instances. So we will not flood the database ;)

Based on https://github.com/boxine/django-huey-monitor/pull/67

codecov-commenter commented 2 years ago

Codecov Report

Merging #68 (3ebd73c) into master (3481a79) will increase coverage by 0.10%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   83.46%   83.56%   +0.10%     
==========================================
  Files          35       35              
  Lines         786      791       +5     
==========================================
+ Hits          656      661       +5     
  Misses        130      130              
Impacted Files Coverage Δ
huey_monitor/tqdm.py 100.00% <100.00%> (ø)
huey_monitor_tests/tests/test_tqdm.py 100.00% <100.00%> (ø)

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 3481a79...3ebd73c. Read the comment docs.

Skrattoune commented 2 years ago

Thanks a lot! I'll test and update now