Skip to content

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

Open
valyo wants to merge 8 commits into
devfrom
improve-project-updating-during-upload
Open

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

Conversation

@valyo

@valyo valyo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

codecov Bot commented Mar 31, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 38.88889% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.29%. Comparing base (df9e463) to head (a104555).

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   55.40%   55.29%   -0.12%     
==========================================
  Files          33       33              
  Lines        3061     3078      +17     
==========================================
+ Hits         1696     1702       +6     
- Misses       1365     1376      +11     

☔ View full report in Codecov by Harness.
📢 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.

@valyo
valyo marked this pull request as ready for review June 10, 2026 09:14
@valyo
valyo requested a review from a team as a code owner June 10, 2026 09:14
@valyo valyo added the type: feature For release template label Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature For release template

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant