Skip to content

Conversation

@ychiucco
Copy link
Collaborator

@ychiucco ychiucco commented Dec 1, 2025

closes #3040

Checklist before merging

  • I added an appropriate entry to CHANGELOG.md
  • I added logging to new code - if appropriate.
  • I merged main into the current branch.

@ychiucco ychiucco linked an issue Dec 1, 2025 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Dec 1, 2025

Benchmark comparison

GET /api/alive/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
vanilla 0.7 0.8 0.87 0.0 0.0
power 0.7 0.9 0.78 0.0 0.0
dataset 0.7 0.8 0.87 0.0 0.0
project 0.7 0.7 1.00 0.0 0.0
job 0.7 0.7 1.00 0.0 0.0

GET /api/v2/project/3/dataset/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
dataset 82.4 79.8 1.03 4.6 4.6

GET /api/v2/job/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
vanilla 4.4 5.3 0.83 2.7 2.7
power 47.7 44.4 1.07 787.3 787.3
dataset 48.3 46.1 1.05 1097.2 1097.2
project 16.3 18.3 0.89 343.8 343.8
job 18.1 13.1 1.38 273.0 273.0

GET /api/v2/project/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
vanilla 4.3 4.5 0.96 0.1 0.1
power 4.1 4.2 0.98 0.1 0.1
dataset 4.1 3.9 1.05 0.1 0.1
project 4.7 4.2 1.12 2.3 2.3
job 4.3 3.8 1.13 0.1 0.1

GET /api/v2/task/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
vanilla 6.0 5.3 1.13 0.8 0.8
power 5.6 4.9 1.14 0.8 0.8
dataset 5.8 4.9 1.18 0.8 0.8
project 5.9 4.9 1.20 0.8 0.8
job 5.8 4.9 1.18 0.8 0.8

GET /api/v2/task-group/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
vanilla 8.1 6.4 1.27 1.8 1.8
power 7.7 6.7 1.15 1.8 1.8
dataset 7.4 6.5 1.14 1.8 1.8
project 7.8 6.9 1.13 1.8 1.8
job 7.9 6.8 1.16 1.8 1.8

GET /api/v2/project/2/workflow/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
power 12.5 10.6 1.18 28.0 28.0

POST /api/v2/project/3/dataset/417/images/query/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
dataset 41.0 42.2 0.97 275.1 275.1

GET /auth/current-user/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
vanilla 3.2 3.2 1.00 0.2 0.2
power 3.0 3.0 1.00 0.2 0.2
dataset 3.0 3.0 1.00 0.2 0.2
project 3.0 2.9 1.03 0.2 0.2
job 2.9 2.9 1.00 0.2 0.2

POST /auth/token/login/

User Time current (ms) Time main (ms) Ratio current/main Size current (Kb) Size main (Kb)
dataset 272.2 271.6 1.00 0.2 0.2

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  fractal_server/app/models/v2
  job.py
  fractal_server/app/routes/api/v2
  _aux_functions.py
  history.py
  status_legacy.py
Project Total  

This report was generated by python-coverage-comment-action

[skip ci]
[skip ci]
[skip ci]
@tcompa tcompa marked this pull request as ready for review December 2, 2025 10:04
@tcompa tcompa merged commit 944d861 into main Dec 2, 2025
@tcompa tcompa deleted the 3040-add-database-constraint-for-jobs-through-partial-index branch December 2, 2025 10:04
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.

Add database constraint for jobs (through partial index)

3 participants