[Actor] Add actor state management#35
Conversation
youngbupark
changed the title
| "operation": MAP_CHANGE_KIND_TO_OPERATION.get(state.change_kind) or '', | ||
| "request": { | ||
| "key": state.state_name, | ||
| "value": base64encoded, |
There was a problem hiding this comment.
Is this saving data the same way as in the Java SDK?
There was a problem hiding this comment.
It is same as dotnet one. I assume dotnet one is compatible with java one.
|
@pruthvidhodda @artursouza @amanbha I addressed all your comments. please review this pr again. |
orizohar
left a comment
orizohar
left a comment
There was a problem hiding this comment.
Aside from the minor requested additions in READMEs, we should start creating a docs directory in this repo and as additional functionality is added, add SDK specific documentation there.
|
@youngbupark - can you please some more context on the main REDME.md file? We need to have a sentence on dapr and links to the dapr/docs and getting started content for users who arrive here with no context. |
I will file the issue to improve README. This pr is mainly for state management implementation. This PR is going to feature branch, not master :) When we merge feature branch to master, I will update README.md |
6 participants
Description
Add actor state management
Issue reference
#23
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: