Skip to content

feat: add multiscan#7

Open
McPatate wants to merge 2 commits into
LevitatingOrange:mainfrom
huggingface:feat/add_multiscan
Open

feat: add multiscan#7
McPatate wants to merge 2 commits into
LevitatingOrange:mainfrom
huggingface:feat/add_multiscan

Conversation

@McPatate

@McPatate McPatate commented Jul 18, 2023

Copy link
Copy Markdown
Collaborator
  • fix: concurrency issues

I was having an issue in production where in a multithreaded context when using ClamdClient with ConnectionType::Oneshot the connections were dropped at every connect call (only the last one calling connect "won").

That was due to a poor quality of coding on my behalf, sorry about that!

* fix: concurrency issues
@McPatate

Copy link
Copy Markdown
Collaborator Author

Still having the same issue with the tests, they work locally :(

@McPatate McPatate force-pushed the feat/add_multiscan branch from 184ad22 to a374dd0 Compare May 5, 2025 13:50
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.

1 participant