WIP: [DEVEX-227] Getting state from events and running busines logic #334
+2,624
−2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nsions They're redundant as most of them should be handled by the ReadStreamOptions. Expected checks can be made on the append side and also passed to decide options to only read as many events as needed.
…tween StateBuilder and AsyncStateBuilder
…g and decision making
Also simplified the StateBuilder abstraction to reduce the number of classes. Reshaped Decider implementation, now AsyncDecider uses raw code and regular Decider wraps around it.
Now AsyncEnumerable part will always assume that it gets the exact initial state, it needs to be read first either from state builder or snapshot. That makes it explicit on the responsibilities.
…re to a dedicated file
…e overloads to extension methods
oskardudycz
force-pushed
the
DEVEX-227-GettingStateAndMakingDecisions
branch
from
d71db29 to
a84083a
Compare
… extension method for regular events
Made also other smaller improvements like: - removing event from state store interface, - made both StateStore and AggregateStore method virtual, - decoupled Decider from AsyncDecider moving the mapping method to extension classes.
… of the single state That gives possibility to cache multiple single stream projection states instead of rolling always a single one
oskardudycz
force-pushed
the
DEVEX-227-GettingStateAndMakingDecisions
branch
from
f69a63a to
dd8cf4c
Compare
oskardudycz
force-pushed
the
DEVEX-227-GettingStateAndMakingDecisions
branch
2 times, most recently
from
8c7d67a to
7b1b13c
Compare
oskardudycz
force-pushed
the
DEVEX-227-GettingStateAndMakingDecisions
branch
from
7b1b13c to
0a5bc23
Compare
…ggregate Store Also split the Decider information and Decision processing
oskardudycz
force-pushed
the
DEVEX-227-GettingStateAndMakingDecisions
branch
from
86c22ab to
a8a4f4a
Compare
oskardudycz
force-pushed
the
DEVEX-227-GettingStateAndMakingDecisions
branch
from
a8a4f4a to
d2f9d4e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚧 WIP