Implement QoS: liveliness, deadline, lifespan #266
Conversation
**Summary** * Add rmw qos interfaces * Implement rmw_wait * Implement rmw_take_event * Reformat comments from pr * Add error checking in rmw_wait and rmw_event Signed-off-by: Ross Desmond <44277324+ross-desmond@users.noreply.github.com>
Signed-off-by: Ross Desmond <44277324+ross-desmond@users.noreply.github.com>
Signed-off-by: Miaofei <miaofei@amazon.com>
Signed-off-by: Miaofei <miaofei@amazon.com>
I'm going to hold off reviewing this one until some of the common nitpick's I pointed out in ros2/rmw_connext#352 (review) can be checked for here too. |
It doesn't look like any commits have been pushed here since my last comment. I think most of the things I mentioned on the
Maybe you guys could take a pass over this before I do a full review? |
Signed-off-by: Miaofei <miaofei@amazon.com>
Hi @wjwwood, the common nitpicks should be corrected now. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few nitpicks, but looks good overall.
Signed-off-by: Miaofei <miaofei@amazon.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Signed-off-by: Miaofei <miaofei@amazon.com>
This PR introduces a few compiler warnings, see https://ci.ros2.org/job/ci_packaging_windows/105/warnings43Result/ Please address them in another PR rather sooner than later. |
Opened another PR for addressing the new compiler warnings at #269. |
Summary
Provide implementation of QoS changes for rmw events. If applied this PR will
Connects to ros2/rmw#171