mirror of
https://github.com/koel/koel.git
synced 2026-04-26 01:06:00 +03:00
[GH-ISSUE #1297] FTS database locking when syncing multiple files simultaneously #745
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#745
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
Originally created by @vitoyucepi on GitHub (Mar 30, 2021).
Original GitHub issue: https://github.com/koel/koel/issues/1297
Describe the bug
If I try adding multiple files at the same time, then some of them can fail.
To reproduce
Steps to reproduce the behavior:
find /path/to/music/dir -type f | xargs -I {} sh -c 'php artisan koel:sync CLOSE_WRITE,CLOSE {}'Expected behavior
No sqlite errors. Maybe increase wait for lock time.
Screenshots
Environment
Additional context
Originally I've found this error by trying to speedup
inotifywait.I think this might happen if I want to add files in parallel.
inotifywaitand fromcronat the same time.@vitoyucepi commented on GitHub (Mar 30, 2021):
This can be triggered via webui.
To reproduce
Steps to reproduce the behavior:
I've tested 10 files. Maybe it depends on how fast I upload files.
@vitoyucepi commented on GitHub (Mar 30, 2021):
Maybe also related issue. Reproduced with default
docker-compose.ymlfrom https://github.com/koel/docker@phanan commented on GitHub (Apr 11, 2021):
I think it's unrelated to Koel, rather TNTSearch as seen in the log—please open an issue there. Even so, I'd like to think that TNTSearch should already be able to handle more than a handful of concurrent updates.
@Zaedus commented on GitHub (Jun 20, 2022):
For those who are getting this error still, I recommend just continuously running
php artisan koel:syncuntil you get all of your songs synced. Slowly but surely it syncs. You can even write a bash script to just keep syncing every time it fails:Make sure to keep an eye on this though! It does not stop until you manually Control-C it.