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

EventDescriptor purpose #10

Open
eagrzn opened this issue Apr 22, 2015 · 4 comments
Open

EventDescriptor purpose #10

eagrzn opened this issue Apr 22, 2015 · 4 comments

Comments

@eagrzn
Copy link

eagrzn commented Apr 22, 2015

Thanks for great code!
I have question: what purpose of internal struct EventDescriptor?
https://github.com/gregoryyoung/m-r/blob/master/SimpleCQRS/EventStore.cs#L17
Why I should store some kind of event envelope instead of event itself?

@gregoryyoung
Copy link
Owner

https://github.com/gregoryyoung/m-r/blob/master/SimpleCQRS/EventStore.cs#L37

There are things not exposed from event?

On Wed, Apr 22, 2015 at 11:42 AM, Evgeniy Gryaznov <notifications@github.com

wrote:

Thanks for great code!
I have question: what purpose of internal struct EventDescriptor?

https://github.com/gregoryyoung/m-r/blob/master/SimpleCQRS/EventStore.cs#L17
Why I should store some kind of event envelope instead of event itself?


Reply to this email directly or view it on GitHub
#10.

Studying for the Turing test

@eagrzn
Copy link
Author

eagrzn commented Apr 23, 2015

Do you mean the aggregate identifier?

@gregoryyoung
Copy link
Owner

Could be other things as well... depends on your implementation
On 23 Apr 2015 09:51, "Evgeniy Gryaznov" notifications@github.com wrote:

Do you mean the aggregate identifier?


Reply to this email directly or view it on GitHub
#10 (comment).

@norvegec
Copy link

norvegec commented Dec 11, 2022

Guess it's clear at this point.
I wonder, though, if Version itself belongs to Event or should be moved to it's metadata.

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