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.

Sign up for GitHub

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

Jump to bottom

Remove the key from commands and events in the StatefulApp API #148

Closed
AttilaMihaly opened this issue Sep 8, 2020 · 0 comments
Closed

Remove the key from commands and events in the StatefulApp API #148

AttilaMihaly opened this issue Sep 8, 2020 · 0 comments
Assignees
@AttilaMihaly
Labels
enhancement New feature or request

Comments

@AttilaMihaly
Copy link
Member

StatefulApp is assumed to be partitioned by a key so passing it around in the business logic brings no benefit and adds accidental complexity.
@AttilaMihaly AttilaMihaly added the enhancement New feature or request label Sep 8, 2020
@AttilaMihaly AttilaMihaly self-assigned this Sep 8, 2020
AttilaMihaly added a commit to AttilaMihaly/morphir-elm that referenced this issue Sep 8, 2020
@AttilaMihaly
Removed key from API and added sample model with tests. finos#148
661a966
AttilaMihaly added a commit that referenced this issue Sep 8, 2020
@AttilaMihaly
Removed key from API and added sample model with tests. #148 (#149)
1e954a0 
* Removed key from API and added sample model with tests. #148

* Fixed resolution for StatefulApp.

* Fix merge error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AttilaMihaly AttilaMihaly

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@AttilaMihaly