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.

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

rcl_send_response returns RCL_RET_TIMEOUT. #1080

Closed
wants to merge 1 commit into from

Conversation

kghost
Copy link

@kghost kghost commented Jun 16, 2023

part of ros2/ros2#1253

We hit this problem where a broken client crashes the service executor. With this change we can fix it in rclcpp.

Note: this is same as #1048 which is not taken attention for a while.

@kghost kghost force-pushed the rolling branch 3 times, most recently from 0ab9550 to ba9ceea Compare June 16, 2023 02:16
part of ros2/ros2#1253

We hit this problem where a broken client crashes the service executor.
With this change we can fix it in rclcpp.

Note: this is same as ros2#1048 which is not been taken attention for a
while.

Signed-off-by: Zang MingJie <zealot0630@gmail.com>
Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

as you mentioned, this is duplication of #1048. you can review #1048 instead?

@emersonknapp
Copy link
Collaborator

Closing as duplicate of #1048 - @fujitatomoya is there anything blocking us from moving forward on that one?

@fujitatomoya
Copy link
Collaborator

@emersonknapp see ros2/rclcpp#2215 (comment), i do not figure out what/how we should do on client with this return code yet. that is the reason i was hesitating to merge #1048... any ideas? probably we want to keep the discussion on ros2/rclcpp#2215 to track.

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.

3 participants