Skip to content

Fix/old sqlite docker#568

Merged
g0ldyy merged 2 commits into
developmentfrom
fix/old-sqlite-docker
Mar 21, 2026
Merged

Fix/old sqlite docker#568
g0ldyy merged 2 commits into
developmentfrom
fix/old-sqlite-docker

Conversation

@g0ldyy

@g0ldyy g0ldyy commented Mar 21, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes
    • Improved reliability of database migration queries for metadata and search tracking tables.

@g0ldyy g0ldyy merged commit a0c75e5 into development Mar 21, 2026
2 checks passed
@g0ldyy g0ldyy deleted the fix/old-sqlite-docker branch March 21, 2026 15:38
@coderabbitai

coderabbitai Bot commented Mar 21, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e90df458-b406-4e40-930b-dee4cdc7cda7

📥 Commits

Reviewing files that changed from the base of the PR and between 70a2758 and 41874cb.

📒 Files selected for processing (1)
  • comet/core/schema_migrations.py

Walkthrough

Added unconditional WHERE 1=1 clauses to three INSERT...SELECT...ON CONFLICT DO UPDATE backfill queries in schema migrations for metadata_cache, digital_release_cache, and first_searches tables.

Changes

Cohort / File(s) Summary
Schema Migration Backfill Updates
comet/core/schema_migrations.py
Added WHERE 1=1 clauses to three INSERT...SELECT...ON CONFLICT DO UPDATE queries in the _migration_backfill_canonical_tables function for metadata caching, release date, and search tracking tables.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.

OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required.

@coderabbitai coderabbitai Bot mentioned this pull request Apr 26, 2026
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.

1 participant