Optimize CreateClassifications

1000

This does not fix the memory leak (#458 (closed)), but it makes the endpoint faster, halves the SQL queries needed in the worst cases, and more in line with the way we write bulk endpoints now.

added 13 commits

• 7f2c85b7...393d8e33 - 11 commits from branch master
• b73299f9 - Avoid an extra query for corpora
• 32d72b3d - Lower the amount of SQL queries in CreateClassifications

Compare with previous version

assigned to @babadie and unassigned @erwanrouchet

changed the description

Codecov Report

Merging #1000 into master will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1000   +/-   ##
=======================================
  Coverage   88.09%   88.10%           
=======================================
  Files         230      230           
  Lines       15196    15206   +10     
=======================================
+ Hits        13387    13397   +10     
  Misses       1809     1809           

Impacted Files	Coverage Δ
arkindex/documents/api/ml.py	97.76% <ø> (-0.07%)
arkindex/documents/serializers/ml.py	100.00% <100.00%> (ø)
...kindex/documents/tests/test_bulk_classification.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 623e728...9cd9329. Read the comment docs.

added 34 commits

• 32d72b3d...623e7280 - 32 commits from branch master
• 4f31fa28 - Avoid an extra query for corpora
• 9cd9329b - Lower the amount of SQL queries in CreateClassifications

Compare with previous version

mentioned in commit 497635f1

merged