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

When does CycloneDDS trigger message lost event? #642

Open
mhaukeland opened this issue Jun 19, 2023 · 2 comments
Open

When does CycloneDDS trigger message lost event? #642

mhaukeland opened this issue Jun 19, 2023 · 2 comments
Labels
more-information-needed Further information is required

Comments

@mhaukeland
Copy link

In the quality of service demo it is stated that: "CycloneDDS partially implements the feature (message lost event) and it only triggers the event under some limited circumstances" Can anyone answer when CycloneDDS actually triggers the message lost event? And maybe even compare it to the FASTRTPS_DDS or the ConnextDDS?

@clalancette
Copy link
Contributor

@eboasson Could you take a quick look and see if this is a question you could answer? Thanks.

@clalancette clalancette added the more-information-needed Further information is required label Jun 29, 2023
@eboasson
Copy link

@clalancette happy to.

@mhaukeland I can't compare it to FastDDS or Connext, for the simple reason that I don't know those implementations well enough. What I do know is that Cyclone DDS currently does not report sample lost events, and that this is a known issue.

Note that the issue is not detecting sample lost events, as deciding to skip sequence numbers is quite fundamental to the protocol stack. So the issue is really just setting the status/invoking the listener when it happens.

I have a number of reasons for wanting to fix it soon, but I have to admit that I sometimes stretch the concept of "soon" to the breaking point ... Please ping me if you don't hear something about it soon so I don't forget 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
more-information-needed Further information is required
Projects
None yet
Development

No branches or pull requests

3 participants