Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Dashing] Fix hang with timers in MultiThreadedExecutor (#835) #869

Merged
merged 1 commit into from
Sep 25, 2019
Merged

[Dashing] Fix hang with timers in MultiThreadedExecutor (#835) #869

merged 1 commit into from
Sep 25, 2019

Conversation

ivanpauno
Copy link
Member

Backports #836.

@malsbat @ivanpauno
Fix hang with timers in MultiThreadedExecutor (#835) (#836)
2b7d012 
Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
@ivanpauno ivanpauno added the in review Waiting for review (Kanban column) label Sep 24, 2019
@ivanpauno ivanpauno self-assigned this Sep 24, 2019
@ivanpauno ivanpauno mentioned this pull request Sep 24, 2019
Closed
@ivanpauno
Copy link
Member Author

CI, up to rclcpp test_rclcpp, dashing repos:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno ivanpauno merged commit 895145c into dashing Sep 25, 2019
@clalancette clalancette deleted the ivanpauno/backport-#836 branch October 14, 2019 17:14
@ivanpauno ivanpauno mentioned this pull request May 29, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wjwwood wjwwood wjwwood approved these changes

Assignees

@ivanpauno ivanpauno

Labels
in review Waiting for review (Kanban column)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ivanpauno @wjwwood @malsbat