Skip to content

Add a call to project updating endpoind at the end of an upload#942

Draft
valyo wants to merge 3 commits intodevfrom
improve-project-updating-during-upload
Draft

Add a call to project updating endpoind at the end of an upload#942
valyo wants to merge 3 commits intodevfrom
improve-project-updating-during-upload

Conversation

@valyo
Copy link
Copy Markdown
Member

@valyo valyo commented Mar 31, 2026

Pull Request Template

Before Marking as Ready for Review

  • Add relevant information to the sections below (Summary etc)
  • Rebase or merge the latest dev (or other targeted branch)
  • Update documentation if needed
  • Add an entry to the SPRINTLOG.md if needed
  • Choose an appropriate label. See here for information on the labelling options
  • The code follows the style guidelines
  • Perform a self-review: read the diff as if reviewing someone else's code
  • I have commented the code, particularly in hard-to-understand areas
  • Verify that all checks and tests have passed

If the target branch is master:

Summary

Backend was changed so that uploads no longer
touch the projects row for each file, to reduce lock contention under concurrent uploads.

Endpoint for project update at the end of an upload is added

Load testing shows that the number of timeouts is reduced by about 50% all other conditions equal.

Related Issue/Ticket

dds_web #1813

Testing

If applicable: How did you verify the change? Include commands, data, or screenshots.

Reviewer Notes

Anything that helps reviewers (e.g. areas needing close attention).


Once all boxes are checked, mark the PR as Ready for Review and tag at least one team member as the initial reviewer.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 31, 2026

Codecov Report

❌ Patch coverage is 38.88889% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.34%. Comparing base (b488abc) to head (9d42b22).

Files with missing lines Patch % Lines
dds_cli/data_putter.py 31.25% 11 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #942      +/-   ##
==========================================
- Coverage   54.45%   54.34%   -0.11%     
==========================================
  Files          33       33              
  Lines        3043     3060      +17     
==========================================
+ Hits         1657     1663       +6     
- Misses       1386     1397      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant