Skip to content

[DOP-38392] Use recursive symlink queries#478

Merged
dolfinus merged 1 commit into
developfrom
feature/DOP-38392
Jul 4, 2026
Merged

[DOP-38392] Use recursive symlink queries#478
dolfinus merged 1 commit into
developfrom
feature/DOP-38392

Conversation

@IlyasDevelopment

@IlyasDevelopment IlyasDevelopment commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Change Summary

Reworked dataset symlink reads to use recursive CTEs over dataset_symlink_group.

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.

@IlyasDevelopment IlyasDevelopment self-assigned this Jul 3, 2026
@IlyasDevelopment IlyasDevelopment added the ci:skip-changelog Add this label to skip changelog file check label Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissBranchBrPartCoverMissing
data_rentgen/server/services
   lineage.py507222121195%111, 129, 287, 318, 497, 527, 663, 694, 757->767, 767–769, 784, 886->885, 889->888, 1105->1104, 1108->1107, 1186, 1251, 1272->1274, 1280->1282, 1331->1339, 1339, 1341–1344, 1386->1388, 1388–1390
TOTAL7700517130221093% 

Comment thread data_rentgen/db/models/dataset_symlink_group.py Outdated
Comment thread data_rentgen/db/repositories/dataset_symlink.py Outdated
Comment thread data_rentgen/db/repositories/job_dependency.py Outdated
@IlyasDevelopment IlyasDevelopment force-pushed the feature/DOP-38392 branch 2 times, most recently from 227983a to 475f159 Compare July 3, 2026 15:35
Comment thread data_rentgen/db/repositories/dataset_symlink.py Outdated
Comment thread data_rentgen/db/repositories/dataset_symlink.py Outdated
@dolfinus dolfinus merged commit c5186b4 into develop Jul 4, 2026
11 checks passed
@dolfinus dolfinus deleted the feature/DOP-38392 branch July 4, 2026 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip-changelog Add this label to skip changelog file check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants