roadrunner-php / laravel-bridge

🌉 RoadRunner ⇆ Laravel bridge 🇺🇦❤️
https://roadrunner.dev/docs/integration-laravel
MIT License
372 stars 25 forks source link

Использование воркеров под разные плагины RR #61

Closed butschster closed 3 years ago

butschster commented 3 years ago

Насколько я понимаю сейчас RR может работать только с HTTP и если, допустим, мне необходимо добавить TCP плагин, то мне необходимо для него использовать альтернытивный воркер со своим обработчиком TCP запросов

tarampampam commented 3 years ago

It depends on your plugin code, of course. Issue can be closed?

butschster commented 3 years ago

Нет. Скорее всего пакет для Laravel необходимо будет расширять, например добавить воркеры для очередей и стоит посмотреть в сторону разделения воркеров на разные типы. Сейчас есть только https://github.com/spiral/roadrunner-laravel/blob/master/src/Worker.php, скорее всего он должен стать HttpWorker и добавиться еще QueueWorker и т.д.

tarampampam commented 3 years ago

Please, use English for your messages (this is important for the community).

When different types of workers will be implemented - existing classes will be renamed respectively :smiley:

butschster commented 3 years ago

Please, use English for your messages (this is important for the community).

When different types of workers will be implemented - existing classes will be renamed respectively 😃

Right now we have jobs, http

tarampampam commented 3 years ago

At this moment this package supports only HTTP workers (and nothing more). If you want to:

Sorry, but at this moment I can't understand what exactly you want :(

butschster commented 3 years ago

В общем эта issue была для @SerafimArts чтобы подумать как быть в том случае, если разработчик захочет не только HTTP использовать, а еще Jobs. В будущем может появиться еще вариации воркеров и необходимо предусмотреть возможность расширения, т.к. сейчас пакет завязан исключительно на HTTP воркер. Чтобы расширить поведение мне сейчас пришлось копировать код и добавлять туда дополнительную реализацию. Пример можно посмотреть здесь https://github.com/buggregator/app/blob/feature/roadrunner/bin/rr-worker#L71

tarampampam commented 3 years ago

Oh, ok, I think I understand you now :) Can I close this issue and continue our discussion in #62?

tarampampam commented 3 years ago

ping @butschster

butschster commented 3 years ago

Oh, ok, I think I understand you now :) Can I close this issue and continue our discussion in #62?

I think yes, we can close the issue.