fix database locked error

[jonaswinkler]

Proposed change

Changing the owner on many documents (lets say 2000+) takes a while. With SQLite, "Database locked" errors can be observed, since both the main thread changing permissions as well as the running celery task changing owners run at the same time. Updating owners is actually very fast and doesn't require a separate task.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Other (please explain)

Checklist:

• I have read & agree with the contributing guidelines.
• If applicable, I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
• If applicable, I have checked that all tests pass, see documentation.
• I have run all pre-commit hooks, see documentation.
• I have made corresponding changes to the documentation as needed.
• I have checked my modifications for any breaking changes.

[codecov]

Codecov Report

Merging #2808 (c87ca25) into dev (3dbb7e5) will increase coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev    #2808      +/-   ##
==========================================
+ Coverage   93.04%   93.14%   +0.09%     
==========================================
  Files         149      149              
  Lines        6395     6386       -9     
==========================================
- Hits         5950     5948       -2     
+ Misses        445      438       -7     

Flag	Coverage Δ
backend	93.14% <0.00%> (+0.09%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/documents/bulk_edit.py	86.48% <0.00%> (-0.19%)	⬇️
src/documents/tasks.py	92.30% <ø> (+2.67%)	⬆️
src/paperless_mail/parsers.py	92.61% <0.00%> (+0.98%)	⬆️

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

[jonaswinkler]

I disagree with the failing check, I've actually increased coverage by removing uncovered lines :)

[stumpylog]

I think you're both right. This increases coverage, but the changed lines aren't covered (but I don't think they were before). I'm fine ignoring it in this case.

Actual change makes sense, it should be a quick operation

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.