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

User setting for Qos depth is ignored while using Clinet/Service #339

Open
Barry-Xu-2018 opened this issue Sep 28, 2021 · 2 comments
Open
Labels
help wanted Extra attention is needed

Comments

@Barry-Xu-2018
Copy link
Contributor

This problem was reported at ros2/rclcpp#1785.

After checking code, find

dds_qset_history(qos, DDS_HISTORY_KEEP_ALL, DDS_LENGTH_UNLIMITED);

This line of code causes user settings to be ignored.
Is there a reason for writing the code like this ? Or this is a bug ?

@eboasson
Copy link
Collaborator

It is a bug (see ros2/rclcpp#1785 (comment) for more info).

@Barry-Xu-2018
Copy link
Contributor Author

It is a bug (see ros2/rclcpp#1785 (comment) for more info).

Thanks for your comments.

Based on the fact that the user should be responsible for his actions, if the user doesn't configure, we can keep the current default setting.
If the user wants to configure QoS, we accept user setting.

@clalancette clalancette added the help wanted Extra attention is needed label Oct 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants