-
Notifications
You must be signed in to change notification settings - Fork 2k
Fomo framework for PHP #9584
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fomo framework for PHP #9584
Conversation
frameworks/PHP/fomo/.env
Outdated
| @@ -0,0 +1,22 @@ | |||
| APP_NAME=Fomo | |||
| APP_TIMEZONE=UTC | |||
| APP_WORKER_COUNT=8 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete APP_WORKER_COUNT, so the server will adapt dynamically to the computer CPUs.
The bench run with different servers machines, the citrine actually with 28 cores and 56 threads.
In server.php
'additional' => [
'worker_num' => env('APP_WORKER_COUNT' , cpuCount() * 2) ,
✋ @n0nag0n
| @@ -0,0 +1,22 @@ | |||
| FROM phpswoole/swoole:5.1.3-php8.3 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use php8.4, than swoole have the docker image ready.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Work with PHP8.4, but with deprecated errors.
https://github.com/joanhey/FrameworkBenchmarks/actions/runs/13224796813/job/36914299789#step:9:139
| Fomo\Services\Auth::class , | ||
| Fomo\Services\Cache::class , | ||
| Fomo\Services\Database::class , | ||
| Fomo\Services\Elasticsearch::class , | ||
| Fomo\Services\Language::class , | ||
| Fomo\Services\Mail::class , | ||
| Fomo\Services\Redis::class , | ||
| Fomo\Services\Response::class , | ||
| Fomo\Services\Validation::class , |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove unnecessary services: mail, redis, elasticsearch, ...
And the corresponding config files.
Work but with deprecated errrors.
Update Fomo framework
|
@joanhey Thanks for cleaning that up. I had messed with it last October and forgot to merge it in here. I know when I messed with it before, removing those config problems caused fatal errors (believe it or not), but maybe things have been fixed since then? |
|
Remember to show the deprecated errors in Fomo ?? |
Adding new framework, Fomo Framework based on Swoole backend.
Referenced here #7904