Неправильное поведение при повторном вызове table_sync:work
@evgeek is already working on this.
Since May 26, 2023.
Assessment
This issue has not been assessed yet.
Description
Если у вас в фоне запущен воркер table_sync:work, и вы повторно вызовете table_sync:work, то увидите ошибку, что воркер уже запущен. Но тогда сработает деструктор:
https://github.com/umbrellio/php-table-sync/blob/4e415f67df9be49273508880148691af17663324/src/Integration/Laravel/Console/Commands/WorkCommand.php#L15-L18
Он удалит pid и третий вызов table_sync:work позволит создать нового воркера, а за первым уже никак нельзя будет следить.
Предлагаю убрать в деструкторе удаление пида. Он должен удаляться только в конце handle() в finally.
Также, возможно, стоит задуматься о том, чтобы сделать поддержку работы нескольких воркеров.
- Dominant language
- PHP
- Stars
- 5
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Similar issues
-
Toolkit
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
[Bug] v-quick-install-app install crashes with ValueError if no supported PHP version is installedOpen
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
langgenius/dify#43074 · 1 comment · 1 reaction ·
Maintainers usually reply within 1 day
-
module: remove unused css
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
backdrop-contrib/field_group#99 · 2 comments ·