find a better fix for late subscription handle destruction
Open
Nobody has claimed this yet.
bug
- Dominant language
- C++
- Stars
- 805
- Forks
- 564
- Avg merge
- 1d 17h
- Merged PRs (30d)
- 27
Description
Basically we should revert https://github.com/ros2/rclcpp/pull/505 and find a better solution.
My current best guess is that the intra process manager is causing this by holding on to the subscription handle until after the node handle is getting destroyed.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing PR #505 and the intra process manager, which the issue identifies as a possible cause of the subscription handle outliving the node handle. Determine a better lifecycle solution than reverting the existing change, with completion shown by eliminating the late subscription handle destruction problem.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- robotics
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100