Skip to content

Slot ThreadPool [14767] - #241

Merged
rsanchez15 merged 13 commits into
mainfrom
feature/thread_pool_dev
Jun 24, 2022
Merged

Slot ThreadPool [14767]#241
rsanchez15 merged 13 commits into
mainfrom
feature/thread_pool_dev

Conversation

@jparisu

@jparisu jparisu commented Jun 17, 2022

Copy link
Copy Markdown
Contributor

This PR adds #234 and merge it with #240

@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:08 Inactive
@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:08 Inactive
@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:08 Inactive
@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:15 Inactive
@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:15 Inactive
@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:15 Inactive
@codecov

codecov Bot commented Jun 17, 2022

Copy link
Copy Markdown

Codecov Report

Merging #241 (ca1428a) into main (ca1428a) will not change coverage.
The diff coverage is n/a.

❗ Current head ca1428a differs from pull request most recent head 0739795. Consider uploading reports for the commit 0739795 to get more accurate results

@@           Coverage Diff           @@
##             main     #241   +/-   ##
=======================================
  Coverage   59.17%   59.17%           
=======================================
  Files          91       91           
  Lines        3165     3165           
  Branches     1144     1144           
=======================================
  Hits         1873     1873           
  Misses        598      598           
  Partials      694      694           

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 ca1428a...0739795. Read the comment docs.

@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:28 Inactive
@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:28 Inactive
@jparisu
jparisu temporarily deployed to codecov June 17, 2022 10:28 Inactive
@jparisu
jparisu force-pushed the feature/thread_pool_dev branch from 389419c to ae6343d Compare June 20, 2022 06:48
@jparisu
jparisu temporarily deployed to codecov June 20, 2022 06:48 Inactive
@jparisu
jparisu temporarily deployed to codecov June 20, 2022 06:48 Inactive
@jparisu
jparisu temporarily deployed to codecov June 20, 2022 06:48 Inactive
@jparisu jparisu mentioned this pull request Jun 20, 2022
3 tasks
@rsanchez15
rsanchez15 temporarily deployed to codecov June 21, 2022 08:06 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov June 21, 2022 08:06 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov June 21, 2022 08:06 Inactive
@rsanchez15
rsanchez15 force-pushed the feature/thread_pool_dev branch from d53124e to 6934740 Compare June 21, 2022 08:07
@rsanchez15
rsanchez15 temporarily deployed to codecov June 21, 2022 08:07 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov June 21, 2022 08:07 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov June 21, 2022 08:07 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov June 22, 2022 06:24 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov June 22, 2022 06:24 Inactive
@rsanchez15
rsanchez15 temporarily deployed to codecov June 22, 2022 06:24 Inactive
@jparisu
jparisu force-pushed the feature/thread_pool_dev branch from 6934740 to 276edea Compare June 23, 2022 10:56
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 10:56 Inactive
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 12:41 Inactive
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 12:48 Inactive
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 12:48 Inactive
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 13:10 Inactive
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 13:10 Inactive
Comment thread docs/rst/notes/forthcoming_version.rst Outdated
Comment thread docs/rst/notes/forthcoming_version.rst Outdated
Comment thread ddsrouter_core/src/cpp/communication/Track.cpp Outdated
Comment thread ddsrouter_core/src/cpp/core/DDSRouterImpl.cpp Outdated
Comment thread docs/rst/user_manual/configuration.rst Outdated
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 14:00 Inactive
@jparisu
jparisu force-pushed the feature/thread_pool_dev branch from 301f101 to 3e65cf9 Compare June 23, 2022 14:22
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 14:22 Inactive
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 14:22 Inactive
jparisu and others added 13 commits June 23, 2022 17:25
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: Raúl Sánchez-Mateos <raul@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu
jparisu force-pushed the feature/thread_pool_dev branch from 3e65cf9 to 0739795 Compare June 23, 2022 15:25
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 15:25 Inactive
@jparisu
jparisu temporarily deployed to codecov June 23, 2022 15:25 Inactive

@rsanchez15 rsanchez15 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@rsanchez15
rsanchez15 merged commit ce4904f into main Jun 24, 2022
@rsanchez15
rsanchez15 deleted the feature/thread_pool_dev branch June 24, 2022 06:17
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.

2 participants