Skip to content

Fix HTTP save state transaction.#516

Merged
artursouza merged 1 commit into
dapr:masterfrom
artursouza:fix_state_trx
Mar 17, 2021
Merged

Fix HTTP save state transaction.#516
artursouza merged 1 commit into
dapr:masterfrom
artursouza:fix_state_trx

Conversation

@artursouza

Copy link
Copy Markdown
Contributor

Description

Fix HTTP save state transaction.

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: #514

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

@codecov

codecov Bot commented Mar 17, 2021

Copy link
Copy Markdown

Codecov Report

Merging #516 (788eea6) into master (9c5084b) will increase coverage by 0.16%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #516      +/-   ##
============================================
+ Coverage     80.43%   80.59%   +0.16%     
- Complexity      974      975       +1     
============================================
  Files            88       88              
  Lines          3015     3015              
  Branches        330      330              
============================================
+ Hits           2425     2430       +5     
+ Misses          424      420       -4     
+ Partials        166      165       -1     
Impacted Files Coverage Δ Complexity Δ
...k/src/main/java/io/dapr/client/DaprClientHttp.java 91.78% <100.00%> (+1.71%) 96.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 9c5084b...788eea6. Read the comment docs.

@beiwei30 beiwei30 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@artursouza artursouza merged commit 6c90286 into dapr:master Mar 17, 2021
artursouza added a commit that referenced this pull request Mar 17, 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.

Potential issue in io.dapr.client.DaprClientHttp#executeStateTransaction

3 participants