rclcpp icon indicating copy to clipboard operation
rclcpp copied to clipboard

Apply expire thread fix for events executors from jazzy to rolling

Open skyegalaxy opened this issue 1 month ago • 1 comments

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.

rolling_without_expire_thread

With the relevant commits cherry-picked, here's the results:

rolling_with_expire_thread

Is this user-facing behavior change?

Did you use Generative AI?

Additional Information

skyegalaxy avatar Dec 20 '25 02:12 skyegalaxy