Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

Improve thread pool #571

Merged
merged 6 commits into from
Oct 31, 2023
Merged

Improve thread pool #571

merged 6 commits into from
Oct 31, 2023

Conversation

MBkkt
Copy link
Contributor

@MBkkt MBkkt commented Oct 30, 2023

No description provided.

@MBkkt MBkkt marked this pull request as ready for review October 30, 2023 12:40
core/utils/async_utils.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@Dronplane Dronplane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MBkkt MBkkt merged commit b37f93e into master Oct 31, 2023
1 check passed
@MBkkt MBkkt deleted the chore/improve-thread-pool branch October 31, 2023 20:07
Dronplane pushed a commit that referenced this pull request Nov 21, 2023
* WIP

* WIP

* WIP

* WIP

* Update core/utils/async_utils.cpp
Dronplane added a commit that referenced this pull request Nov 23, 2023
* Feature/use nonpriority pool (#570)

* new thread pool

* fix

* wip

* add static

* attempt to fix compile warning (#569)

the compile option `-fsized-deallocation` should only be used for C++
code, but not for C code. otherwise clang warns with the following
message:
```
warning: command-line option ‘-fsized-deallocation’ is valid for C++/ObjC++ but not for C
```

* Improve thread pool (#571)

* WIP

* WIP

* WIP

* WIP

* Update core/utils/async_utils.cpp

* Separate WaitGroup (#572)

* Separate WaitGroup

* Update core/utils/wait_group.hpp

Co-authored-by: Andrei Lobov <andrei.lobov@arangodb.com>

* Exact count

* fix lints

---------

Co-authored-by: Andrei Lobov <andrei.lobov@arangodb.com>

* Backport and start

* fix

* fix datarace

* fix build

---------

Co-authored-by: Jan <jsteemann@users.noreply.github.com>
Co-authored-by: Valery Mironov <32071355+MBkkt@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants