broadway / read-model-elasticsearch

Elasticsearch read model implementation using elastic/elasticsearch-php

Broadway

Broadway is a project providing infrastructure and testing helpers for creating CQRS and event sourced applications. Broadway tries hard to not get in your way. The project contains several loosely coupled components that can be used together to provide a full CQRS\ES experience.

About

Read the blog post about this repository at:

• http://labs.qandidate.com/blog/2014/08/26/broadway-our-cqrs-es-framework-open-sourced/

Installation

$ composer require broadway/broadway

Documentation

You can find detailed documentation of the Broadway bundle on broadway.github.io/broadway.

Feel free to join #qandidate on freenode with questions and remarks!

Acknowledgements

The broadway project is heavily inspired by other open source project such as AggregateSource, Axon Framework and Ncqrs.

We also like to thank Benjamin, Marijn and Mathias for the conversations we had along the way that helped us shape the broadway project. In particular Marijn for giving us access to his in-house developed CQRS framework.

License

MIT, see LICENSE.

read-model-elasticsearch's Issues

Bundle is incompatible with version of broadway bundle 0.4.2

Hello,
When I try to configure this bundle with "broadway/broadway-bundle": "^0.4.2", I get an error:
Fatal error: Declaration of Broadway\ReadModel\ElasticSearch\ElasticSearchRepository::save(Broadway\ReadModel\Identifiable $data) must be compatible with Broadway\ReadModel\Repository::save(Broadway\ReadModel\Identifiable $data): void in /var/www/symfony/vendor/broadway/read-model-elasticsearch/src/ElasticSearchRepository.php on line 24

[RFC] create alias when creating Elasticsearch indices

see broadway/broadway#238

Problems using _type with index templates and other use cases

Hey team!

First of all, thank you for your work here, it's a great library and has helped me a lot.

Just wanted to ask if there are any plans from your side to remove the _type field from the repository (as it creates trouble starting in ES 7.0) and it will disappear soon (as I'm sure you already know).

I did some changes in a fork (basically just removed the references to 'type' all over the repository and created a custom serializer to be able to use a _class field inside the payload as a reference instead the type field. It works for me although I'm pretty sure there is a better solution for this case so, I'm asking:

• If you already have any plans to fix this, please let me know, I'm happy to take that task myself if you give me some guidance in what you think is the right implementation.

Thank you and keep the good work!

Be able to set a datatype to each fields for index mapping

It seems like when you create an index inElasticSearchRepository.php type of each non analyzed fields is always string.

read-model-elasticsearch/src/ElasticSearchRepository.php

Lines 286 to 289 in a5a548d

	$fields[$field] = [
	'type' => 'string',
	'index' => 'not_analyzed'
	];

Could be nice to define which type to set with each non analyzed field.