Skip to content

Resolve Issue #37: Remove the sender thread - #86

Merged
EndrII merged 2 commits into
mainfrom
task_37
Oct 11, 2026
Merged

EndrII merged 2 commits into
mainfrom
task_37

Conversation

@QuasarAppCI

Copy link
Copy Markdown
Member

Closes #37. Implemented automatically by Quasar agent.

All sockets and the data sender now work in the node worker (main) thread instead of a separate QThread. AbstractNode::stop() now pumps the main-thread event loop while waiting for pool workers so queued socket jobs can complete (prevents deadlock).
@EndrII EndrII self-assigned this Oct 11, 2026
@EndrII
EndrII self-requested a review October 11, 2026 07:55
@EndrII EndrII removed their assignment Oct 11, 2026

@EndrII EndrII 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.

looks good for me

@EndrII
EndrII enabled auto-merge (squash) October 11, 2026 15:51
Comment thread src/public/async.h Outdated
just for trigger ci

Signed-off-by: Andrei Yankovich <endriipersonal@gmail.com>
@EndrII
EndrII merged commit edefacb into main Oct 11, 2026
3 checks passed
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.

Remove the sender thread.

2 participants