Skip to content

[DOP-33859] Fix case-insensitive fetch by name#486

Merged
dolfinus merged 1 commit into
developfrom
bugfix/DOP-33859
Jul 9, 2026
Merged

[DOP-33859] Fix case-insensitive fetch by name#486
dolfinus merged 1 commit into
developfrom
bugfix/DOP-33859

Conversation

@dolfinus

@dolfinus dolfinus commented Jul 8, 2026

Copy link
Copy Markdown
Member

Change Summary

Make filtering, sorting and locking by lowercase Dataset.name, Job.name, Tag.name, TagValue.value - all of these have lowercase unique index.

Related issue number

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
  • mddocs/docs/changelog/next_release/<pull request or issue id>.<change type>.md file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@dolfinus
dolfinus force-pushed the bugfix/DOP-33859 branch from cce37a1 to 7f9f76e Compare July 8, 2026 17:06
@dolfinus
dolfinus requested review from IlyasDevelopment and TiGrib and removed request for TiGrib July 8, 2026 17:08
@dolfinus dolfinus added the ci:skip-changelog Add this label to skip changelog file check label Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissBranchBrPartCoverMissing
data_rentgen/db/repositories
   dataset.py79022298%110->116, 116->118
   job.py109032299%258->264, 264->266
   run.py13429481377%189->195, 285–286, 288–289, 291, 293, 301, 304–305, 307–308, 315–316, 319–322, 325–326, 329, 346–348, 356, 375–377, 379–380
   user.py3214097%60
data_rentgen/server/services
   dataset.py2110095%38
   job.py71314096%83, 134, 231
TOTAL7741518132821792% 

@dolfinus
dolfinus force-pushed the bugfix/DOP-33859 branch from 7f9f76e to cec7056 Compare July 8, 2026 17:10
@dolfinus
dolfinus requested a review from TiGrib July 8, 2026 17:12
@dolfinus
dolfinus marked this pull request as ready for review July 8, 2026 17:12
@dolfinus dolfinus changed the title [DOP-33859] Fix case-insensitive fetch for tags [DOP-33859] Fix case-insensitive fetch by name Jul 8, 2026
@dolfinus
dolfinus force-pushed the bugfix/DOP-33859 branch from cec7056 to 27f87a8 Compare July 8, 2026 17:24
@dolfinus
dolfinus force-pushed the bugfix/DOP-33859 branch from 27f87a8 to 42b2a75 Compare July 8, 2026 17:26
@dolfinus
dolfinus merged commit 8cffe52 into develop Jul 9, 2026
11 checks passed
@dolfinus
dolfinus deleted the bugfix/DOP-33859 branch July 9, 2026 08:26
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