Skip to content

[DOP-34705] add recursive query for job_dependencies#412

Merged
TiGrib merged 8 commits intodevelopfrom
feature/DOP-34705
Mar 19, 2026
Merged

[DOP-34705] add recursive query for job_dependencies#412
TiGrib merged 8 commits intodevelopfrom
feature/DOP-34705

Conversation

@TiGrib
Copy link
Contributor

@TiGrib TiGrib commented Mar 18, 2026

Change Summary

  • Add depth query parameter to GET /v1/jobs/dependencies endpoint to control how many layers of dependency edges are returned (defaults to 1)
  • Rewrite JobDependencyRepository.get_dependencies to use recursive CTEs (WITH RECURSIVE) for depth-limited upstream/downstream traversal, replacing the previous flat query

Related issue number

[DOP-34705]

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@TiGrib TiGrib requested a review from dolfinus March 18, 2026 14:38
@TiGrib TiGrib self-assigned this Mar 18, 2026
@TiGrib TiGrib force-pushed the feature/DOP-34705 branch from 21ceb0d to 634784d Compare March 19, 2026 08:49
@dolfinus dolfinus added the ci:skip-changelog Add this label to skip changelog file check label Mar 19, 2026
@github-actions
Copy link

github-actions bot commented Mar 19, 2026

Coverage

Coverage Report •
FileStmtsMissBranchBrPartCoverMissing
data_rentgen/server/services
   job.py4610097%74
TOTAL7887971123221287% 

@TiGrib TiGrib marked this pull request as ready for review March 19, 2026 08:55
@dolfinus dolfinus removed the ci:skip-changelog Add this label to skip changelog file check label Mar 19, 2026
@TiGrib TiGrib merged commit 94a3eaf into develop Mar 19, 2026
11 checks passed
@TiGrib TiGrib deleted the feature/DOP-34705 branch March 19, 2026 09:30
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.

2 participants