Skip to content

Refactor tracing demo to show A->B->C call stack.#436

Merged
artursouza merged 1 commit into
dapr:masterfrom
artursouza:tracing_demo_abc
Jan 8, 2021
Merged

Refactor tracing demo to show A->B->C call stack.#436
artursouza merged 1 commit into
dapr:masterfrom
artursouza:tracing_demo_abc

Conversation

@artursouza

Copy link
Copy Markdown
Contributor

Description

Refactor tracing demo to show A->B->C call stack.

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: dapr/dapr#2504

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

Comment thread examples/src/main/java/io/dapr/examples/state/StateClient.java Outdated

@yaron2 yaron2 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

@codecov

codecov Bot commented Jan 8, 2021

Copy link
Copy Markdown

Codecov Report

Merging #436 (2374bba) into master (6a44d51) will decrease coverage by 0.15%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #436      +/-   ##
============================================
- Coverage     83.44%   83.28%   -0.16%     
+ Complexity      851      850       -1     
============================================
  Files            74       74              
  Lines          2519     2519              
  Branches        256      256              
============================================
- Hits           2102     2098       -4     
- Misses          296      299       +3     
- Partials        121      122       +1     
Impacted Files Coverage Δ Complexity Δ
sdk/src/main/java/io/dapr/utils/Retry.java 50.00% <0.00%> (-25.00%) 1.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 6a44d51...2374bba. Read the comment docs.

@artursouza artursouza merged commit 9e13586 into dapr:master Jan 8, 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.

the invocation chain is break when using service invoke

2 participants