Skip to content

Dispatch 1302#479

Closed
kgiusti wants to merge 2 commits intoapache:masterfrom
kgiusti:DISPATCH-1302
Closed

Dispatch 1302#479
kgiusti wants to merge 2 commits intoapache:masterfrom
kgiusti:DISPATCH-1302

Conversation

@kgiusti
Copy link
Copy Markdown
Contributor

@kgiusti kgiusti commented Apr 2, 2019

The "multicast" tests are not actually multicast for now - I'll merge more tests there once unsettled mcast is done. It happened to have a simple reproducer for the settlement race this patch addresses.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #479 into master will increase coverage by 0.01%.
The diff coverage is 94.44%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #479      +/-   ##
=========================================
+ Coverage   86.88%   86.9%   +0.01%     
=========================================
  Files          85      85              
  Lines       19224   19215       -9     
=========================================
- Hits        16703   16699       -4     
+ Misses       2521    2516       -5
Impacted Files Coverage Δ
src/message.c 88.42% <ø> (-0.02%) ⬇️
src/container.c 79.77% <100%> (ø) ⬆️
src/router_node.c 94.24% <100%> (-0.09%) ⬇️
src/router_core/transfer.c 91.4% <88.88%> (+0.33%) ⬆️
src/router_core/modules/edge_router/addr_proxy.c 94.04% <0%> (-0.6%) ⬇️
src/router_core/route_tables.c 76.67% <0%> (-0.25%) ⬇️
src/router_core/router_core.c 86.88% <0%> (-0.22%) ⬇️
src/router_core/connections.c 94.76% <0%> (-0.12%) ⬇️
src/router_core/agent_link.c 67.52% <0%> (+0.51%) ⬆️
... and 2 more

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 512449e...f2482b4. Read the comment docs.

@asfgit asfgit closed this in 425edb6 Apr 3, 2019
@kgiusti kgiusti deleted the DISPATCH-1302 branch April 3, 2019 20:03
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.

3 participants