8000 Implement get_actual_qos() for subscriptions by mm318 · Pull Request #455 · ros2/rcl · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Implement get_actual_qos() for subscriptions #455

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

Merged
merged 5 commits into from
Jun 12, 2019

Conversation

mm318
Copy link
Member
@mm318 mm318 commented Jun 7, 2019

Currently, publishers have the get_actual_qos() feature/function. It would make sense for subscriptions to have them, too.

Signed-off-by: Miaofei <miaofei@amazon.com>
Copy link
Member
@ivanpauno ivanpauno left a comment

Choose a reason for hiding this comment

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

LGTM!
Could you add a test for rcl_subscription_get_actual_qos? See https://github.com/ros2/rcl/blob/master/rcl/test/rcl/test_get_actual_qos.cpp.

Signed-off-by: Miaofei <miaofei@amazon.com>
@mm318
Copy link
Member Author
mm318 commented Jun 9, 2019

Added test. Thanks!

Signed-off-by: Miaofei <miaofei@amazon.com>