Skip to content

Merge pull request #504 from abogdanov37/feature_fix-statestore-seria…#513

Merged
artursouza merged 1 commit into
dapr:release-1.0from
artursouza:hotfix_101
Mar 16, 2021
Merged

Merge pull request #504 from abogdanov37/feature_fix-statestore-seria…#513
artursouza merged 1 commit into
dapr:release-1.0from
artursouza:hotfix_101

Conversation

@artursouza

Copy link
Copy Markdown
Contributor

…lization

Fix issue where custom JSOn serializer generates Base64 encoded data in state store

Description

Fixes for handling state store value as JSON for custom serializer.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #503

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

…lization

Fix issue where custom JSOn serializer generates Base64 encoded data in state store
@artursouza artursouza changed the base branch from master to release-1.0 March 15, 2021 04:32
@codecov

codecov Bot commented Mar 15, 2021

Copy link
Copy Markdown

Codecov Report

Merging #513 (59e3198) into release-1.0 (048d934) will increase coverage by 0.13%.
The diff coverage is 33.33%.

Impacted file tree graph

@@                Coverage Diff                @@
##             release-1.0     #513      +/-   ##
=================================================
+ Coverage          80.29%   80.43%   +0.13%     
  Complexity           974      974              
=================================================
  Files                 88       88              
  Lines               3015     3015              
  Branches             331      330       -1     
=================================================
+ Hits                2421     2425       +4     
+ Misses               427      424       -3     
+ Partials             167      166       -1     
Impacted Files Coverage Δ Complexity Δ
...io/dapr/actors/runtime/DaprStateAsyncProvider.java 60.86% <33.33%> (ø) 13.00 <0.00> (-1.00)
sdk/src/main/java/io/dapr/utils/Retry.java 75.00% <0.00%> (+25.00%) 2.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 048d934...59e3198. Read the comment docs.

@artursouza artursouza merged commit 69606d0 into dapr:release-1.0 Mar 16, 2021
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

Successfully merging this pull request may close these issues.

[RECALLED] Wrong value in state store when use custom json serializer

2 participants