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

Sync #3

Merged
merged 1 commit into from
Apr 4, 2019
Merged

Sync #3

merged 1 commit into from
Apr 4, 2019

Conversation

jhdcs
Copy link
Owner

@jhdcs jhdcs commented Apr 4, 2019

…lish when possible (ros2#674)

  • Only setup intraprocess if 'durability' qos policy is 'volatile'.

Signed-off-by: ivanpauno ivanpauno@ekumenlabs.com

  • Skip interprocess publish when only having intraprocess subscriptions.

Signed-off-by: ivanpauno ivanpauno@ekumenlabs.com

  • Add intraprocess configuration option at publisher/subscription level

Signed-off-by: ivanpauno ivanpauno@ekumenlabs.com

  • Use get_actual_qos when setting-up intraprocess. Add test.

Signed-off-by: ivanpauno ivanpauno@ekumenlabs.com

…lish when possible (#674)

* Only setup intraprocess if 'durability' qos policy is 'volatile'.

Signed-off-by: ivanpauno <ivanpauno@ekumenlabs.com>

* Skip interprocess publish when only having intraprocess subscriptions.

Signed-off-by: ivanpauno <ivanpauno@ekumenlabs.com>

* Add intraprocess configuration option at publisher/subscription level

Signed-off-by: ivanpauno <ivanpauno@ekumenlabs.com>

* Use get_actual_qos when setting-up intraprocess. Add test.

Signed-off-by: ivanpauno <ivanpauno@ekumenlabs.com>
@jhdcs jhdcs merged commit c4ea0a7 into jhdcs:master Apr 4, 2019
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.

2 participants