Skip to content

DISPATCH-1475: prevent I/O thread from accessing link during deletion#631

Merged
asfgit merged 1 commit intoapache:masterfrom
kgiusti:DISPATCH-1475
Nov 26, 2019
Merged

DISPATCH-1475: prevent I/O thread from accessing link during deletion#631
asfgit merged 1 commit intoapache:masterfrom
kgiusti:DISPATCH-1475

Conversation

@kgiusti
Copy link
Contributor

@kgiusti kgiusti commented Nov 26, 2019

Fix includes consistently adding link to proper priority slot in
connections links_with_work array.

Fix includes consistently adding link to proper priority slot in
connections links_with_work array.
@kgiusti kgiusti requested a review from ChugR November 26, 2019 16:04
@ChugR
Copy link
Contributor

ChugR commented Nov 26, 2019

This commit works well. I've blasted my setup for a couple of hours, 250,000 reconnects, all with no issue. Ship it!

@asfgit asfgit merged commit 8c5ee1c into apache:master Nov 26, 2019
@kgiusti kgiusti deleted the DISPATCH-1475 branch November 26, 2019 19:38
@codecov-io
Copy link

Codecov Report

Merging #631 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #631      +/-   ##
==========================================
+ Coverage   86.46%   86.52%   +0.05%     
==========================================
  Files          91       91              
  Lines       20626    20628       +2     
==========================================
+ Hits        17834    17848      +14     
+ Misses       2792     2780      -12
Impacted Files Coverage Δ
src/router_core/delivery.c 89.37% <100%> (+0.19%) ⬆️
src/router_core/forwarder.c 93.95% <100%> (ø) ⬆️
src/router_core/connections.c 94.43% <100%> (+0.56%) ⬆️
src/router_core/transfer.c 94.84% <100%> (+1.47%) ⬆️
src/iterator.c 89.34% <0%> (-0.17%) ⬇️
src/router_node.c 93.09% <0%> (-0.12%) ⬇️
src/container.c 81.37% <0%> (+0.18%) ⬆️
src/parse.c 87.86% <0%> (+0.22%) ⬆️
... and 1 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 bab6dbc...8c5ee1c. Read the comment docs.

ChugR pushed a commit to ChugR/qpid-dispatch that referenced this pull request Oct 13, 2022
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.

5 participants