Skip to content

[DOP-33859] Fix case-insensitive fetch for tags#485

Merged
TiGrib merged 1 commit into
developfrom
bugfix/DOP-33859
Jul 8, 2026
Merged

[DOP-33859] Fix case-insensitive fetch for tags#485
TiGrib merged 1 commit into
developfrom
bugfix/DOP-33859

Conversation

@dolfinus

@dolfinus dolfinus commented Jul 8, 2026

Copy link
Copy Markdown
Member

Change Summary

fetch_bulk/get methods of TagRepository/TagValueRepository ignored existing tags/tag_values due to case-sensitive SELECT, while INSERT is actually case-insensitive, leading to a conflict.

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 requested a review from TiGrib July 8, 2026 12:35
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissBranchBrPartCoverMissing
TOTAL7747527133222192% 
report-only-changed-files is enabled. No files were changed during this commit :)

@TiGrib TiGrib merged commit 9bf75f3 into develop Jul 8, 2026
11 checks passed
@TiGrib TiGrib deleted the bugfix/DOP-33859 branch July 8, 2026 14:56
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