-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Create additional packages for drivers #13
Comments
I wonder if we need non-amqp_interop drivers... |
@zhuravljov what do you think? |
What's currently covered by amqp_interop:
That leaves us with:
|
Agreed, sync belongs in yiisoft/yii-queue. |
https://github.com/yiisoft/yii-queue-interop I don't think we need to create DB driver at this point. We can add it later. |
How about the debug & gii modules of Does these also separate into different packages such as |
Same things goes for db-redis for example which currently embeds:
@samdark how would we handle these & the case mentioned by @larryli ? |
I'm not sure how redis driver is different than db in this case - Yii has its own redis extension with own connection - reusing it should be preferred over whatever |
We also need to take care of porting the stomp driver |
Hi @samdark , do you not think we should actually do DB driver first, reason I say so is barrier to entry, most people have LAMP or WAMP, so without installing anything else they should be use yiisoft/queue. What do you think? |
Could be done, yes. yiisoft/db is released so no obstacles. Do you want to handle it? |
Done! |
No, that should be separate repository. |
Alexander, that is fine but like I said I have never checked anything into Github, maybe you can help me by creating an empty repository, maybe we can call it yiisoft/queue-db, and I can take it from there!?... |
https://forum.yiiframework.com/t/additional-packages-for-specific-drivers-storages/125718/2
The text was updated successfully, but these errors were encountered: