Skip to content
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

个人建议 #44

Open
xufei100 opened this issue Jul 18, 2017 · 4 comments
Open

个人建议 #44

xufei100 opened this issue Jul 18, 2017 · 4 comments

Comments

@xufei100
Copy link

这几天准备使用swoole-crontab 替换 linux crontab

在使用过程中主要以下几个问题

  1. 不清晰怎么启动,补上架构图就ok了,作者补上了,也附上我的 https://processon.com/view/link/59698feae4b064b2bffca3cd
  2. 都启动了就是不执行任务,后面找原因是未安装pcre-devel 不能 foreach swoole_table
  3. 启动后报不能 load resource [cache/session] ,后面去读代码,独立服务启动,不能使用php默认session,需要添加配置文件 cache.php ,里面配置 redis缓存或者文件缓存

为了更好的使用,我建议作者可以考虑以下几点

  1. 去掉nginx+fpm 模式,直接使用 swoole 来做 httpserver 提供admin端web服务
  2. 添加docker运行环境,来避免环境搭建问题,使用环境变量来确定运行环境的选择
  3. 添加好的日志模块,方便调试和错误追踪
  4. 添加swoole-framework为子模块或者使用composer

非常感谢作者的付出,我这提供docker模块,可供你参考

https://github.com/LingdianIT-Com/swoole-crontab/tree/lingdian/docker

@mia0x75
Copy link

mia0x75 commented Jul 18, 2017

不错的建议,作者可以考虑优先 1/4

@osgochina
Copy link
Owner

非常感谢你的建议,开源项目需要大家贡献自己的一份力量.我去附上你的架构图

@osgochina
Copy link
Owner

@xufei100 你如果代理里有更好的修改,欢迎你提交pr

@xufei100
Copy link
Author

@osgochina 我对swoole不是太熟,目前只能做小改动,不完善,所以我就没提交pr了,我后面尽量努力回馈社区

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants