Antonio Maria Sanchez Berrocal comments

Results 10 comments of


                                            Antonio Maria Sanchez Berrocal

Use state serializer content type when saving actor states.

Be my guest. Thanks for take it.

Use state serializer content type when saving actor states.

Did I understand correctly that @artursouza will take this PR to the goal? Or Can I still do something? like fixing Integration tests, or documentation?

Use state serializer content type when saving actor states.

> > Did I understand correctly that @artursouza will take this PR to the goal? Or Can I still do something? like fixing Integration tests, or documentation? > > Sorry...

Use state serializer content type when saving actor states.

Dapr side car doesn't seem to like the content type if the state is null. I checked for it. Please review @artursouza

Use state serializer content type when saving actor states.

@mukundansundar did you have time to review this MR?

Use state serializer content type when saving actor states.

I don't see the button "Merge PR", is it because the branch is out of date? or because I don't have access to merge a PR ?

Dapr client.saveStore(..) doesn't honour stateSerializer contentType

My suggestion to solve the problem is to use the defaultStateSerializer contentType when building the [StateRequest](https://github.com/dapr/java-sdk/blob/master/sdk/src/main/java/io/dapr/client/DaprClientGrpc.java#L577). Similarly like if one saves the state using 'saveBulkState' method: ``` Map meta =...