Skip to content

DISPATCH-1540 - Set the pre-settled flag appropriately on a delivery …#658

Closed
ganeshmurthy wants to merge 2 commits intoapache:masterfrom
ganeshmurthy:DISPATCH-1540
Closed

DISPATCH-1540 - Set the pre-settled flag appropriately on a delivery …#658
ganeshmurthy wants to merge 2 commits intoapache:masterfrom
ganeshmurthy:DISPATCH-1540

Conversation

@ganeshmurthy
Copy link
Copy Markdown
Contributor

…and its peers on large streaming messages. This will allow the presettled counters to be updated correctly

…and its peers on large streaming messages. This will allow the presettled counters to be updated correctly
…ed flag on the delivery inside core thread to avoid contention. Also added some comments
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #658 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #658      +/-   ##
==========================================
+ Coverage   86.53%   86.57%   +0.03%     
==========================================
  Files          91       91              
  Lines       20652    20658       +6     
==========================================
+ Hits        17872    17884      +12     
+ Misses       2780     2774       -6
Impacted Files Coverage Δ
src/router_node.c 93.23% <100%> (ø) ⬆️
src/router_core/delivery.c 89.59% <100%> (+0.12%) ⬆️
src/router_core/forwarder.c 93.95% <0%> (-0.47%) ⬇️
src/server.c 86.56% <0%> (+0.12%) ⬆️
src/parse.c 87.97% <0%> (+0.22%) ⬆️
src/router_core/transfer.c 94.14% <0%> (+0.24%) ⬆️
src/router_core/connections.c 94.32% <0%> (+0.33%) ⬆️
src/router_core/route_tables.c 76.17% <0%> (+0.49%) ⬆️

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 655145d...1e5d89d. Read the comment docs.

@asfgit asfgit closed this in 9d3bf33 Jan 15, 2020
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