unkin/jellyfin-ha-src
4
0
Fork 0
Code Issues 15 Pull Requests Packages Projects Releases Wiki Activity

fix(db): materialise the rating list before updating #6

Merged
benvin merged 1 commits from benvin/fix-rating-levels-pg into main 2026-09-12 22:21:31 +10:00
Conversation 0 Commits 1 Files Changed 2
+100 -1
unkin-agent commented 2026-09-12 21:00:27 +10:00
Member
Copy Link
Copy Source

MigrateRatingLevels issued an ExecuteUpdate while the SELECT DISTINCT reader was still open on the same connection. SQLite tolerates that, Npgsql does not, so the AppInitialisation stage aborted before any schema migration committed and every PostgreSQL instance crash-looped on startup.

  • read the distinct ratings into a list before the update loop
  • cover the migration against a real PostgreSQL, with NULL and empty OfficialRating rows present
MigrateRatingLevels issued an ExecuteUpdate while the SELECT DISTINCT reader was still open on the same connection. SQLite tolerates that, Npgsql does not, so the AppInitialisation stage aborted before any schema migration committed and every PostgreSQL instance crash-looped on startup. - read the distinct ratings into a list before the update loop - cover the migration against a real PostgreSQL, with NULL and empty OfficialRating rows present
unkin-agent added 1 commit 2026-09-12 21:00:28 +10:00
fix(db): materialise the rating list before updating
ci/woodpecker/push/ci Pipeline was successful
Details
ci/woodpecker/pr/ci Pipeline was successful
Details
f59ab11c43 
MigrateRatingLevels issued an ExecuteUpdate while the SELECT DISTINCT reader
was still open on the same connection. SQLite tolerates that, Npgsql does not,
so the AppInitialisation stage aborted and every PostgreSQL instance
crash-looped on startup.

- read the distinct ratings into a list before the update loop
- cover the migration against a real PostgreSQL, with NULL and empty ratings
benvin merged commit b183bec94b into main 2026-09-12 22:21:31 +10:00
benvin deleted branch benvin/fix-rating-levels-pg 2026-09-12 22:21:31 +10:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
benvin (BenVincent) unkin-agent (Unkin Agent) unkinben (Ben Vincent)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: unkin/jellyfin-ha-src#6
Delete Branch "benvin/fix-rating-levels-pg"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?