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

Support mongodb for ReadModel #23

Open
Konafets opened this issue Oct 30, 2017 · 3 comments
Open

Support mongodb for ReadModel #23

Konafets opened this issue Oct 30, 2017 · 3 comments

Comments

@Konafets
Copy link
Contributor

Beside InMemory and ElasticSearch for storing the ReadModel it would be nice if this package would also support the nearly published MongoDB implementation which I found here: https://github.com/broadway/read-model-mongodb

I did not tried this out by myself yet.

@LavaToaster
Copy link
Collaborator

The idea behind not including the elastic search driver in the 2.0.0 update was to allow this sort of thing to happen without updating the package, however, I've just realised this is not the case.

The read model drivers in this package solely exist to bootstrap the read model drivers in broadway. Even then, there's no way to pass through your own implementation as the namespace is hardcoded.

🤔

Requires some thinking to get a good solution in. I can either add in the driver and required config, or I can tidy up the code to allow your own implementation to be passed through, or both.

@Konafets
Copy link
Contributor Author

Konafets commented Oct 30, 2017

Why not adding mongodb as an option to config/broadway.php and add a new MongoDB driver to ReadModel/Broadway/Drivers?

@AlessandroMinoccheri
Copy link
Contributor

News about adding MongoDb?

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

No branches or pull requests

3 participants