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

[PLATFORM-573]: [esrs] POC different projectors on same table #100

Conversation

oliverbrowneprima
Copy link
Contributor

Added an example of two aggregates writing to a shared projection, and some convenience
methods to StoreEvent to make that easier to do

https://prima-assicurazioni-spa.myjetbrains.com/youtrack/issue/PLATFORM-573

Also adds Clone impl for StoreEvent, and StoreEvent::map to allow
trivial mapping from StoreEvent<T> to StoreEvent<U> where T: Into<U>
@oliverbrowneprima oliverbrowneprima requested a review from a team September 1, 2022 11:13
@oliverbrowneprima oliverbrowneprima marked this pull request as ready for review September 1, 2022 11:30
@cpiemontese cpiemontese requested review from fabuloso and Fs00 September 1, 2022 12:20
Copy link
Contributor

@cpiemontese cpiemontese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor typos

examples/aggregate_merging/Readme.md Outdated Show resolved Hide resolved
@cpiemontese cpiemontese requested a review from a team September 1, 2022 12:47
Copy link
Member

@matteosister matteosister left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

examples/aggregate_merging/src/projectors.rs Outdated Show resolved Hide resolved
@oliverbrowneprima oliverbrowneprima merged commit c9cd91a into master Sep 2, 2022
@oliverbrowneprima oliverbrowneprima deleted the PLATFORM-573/user-story/esrs-poc-different-projectors-on-same-table-2 branch September 2, 2022 09:20
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

Successfully merging this pull request may close these issues.

3 participants