rclcpp
rclcpp copied to clipboard
Apply expire thread fix for events executors from jazzy to rolling
Description
In the long actions ram usage section of the benchmarks iRobot shared back in September, there was a memory leak identified in rolling for both the EventsExecutor implementations. I did some deeper profiling and found that it was the result of expired goal results never being cleared out.
As it turns out, there were fixes for this merged directly into the jazzy branch back in March, but it appears that they never made their way to rolling.
With the relevant commits cherry-picked, here's the results: