-
Notifications
You must be signed in to change notification settings - Fork 765
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
[15915] Send GAPs correctly when using separate sending #3012
Conversation
@richiprosima please test this |
@richiprosima please test this again |
@richiprosima please test this again |
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.
Other than the warnings on Windows, changes LGTM
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
…e test Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
aba42dc
to
ad24b9e
Compare
@richiprosima please test windows |
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
@richiprosima Please test this |
@richiprosima Please test mac |
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 with green CI
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
@richiprosima please test mac |
@richiprosima Please test this |
@Mergifyio backport 2.7.x 2.6.x |
* Refs #15915: Added test and error trace Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Disabled for intraprocess Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Added new gap requirement check when using separate sending Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Changed the test from a parametrized test to a standalone test Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Added review suggestions Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Fixed Windows warnings Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Added doxygen. Re-added removed assertion Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Removed unused variables Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> (cherry picked from commit 0344ce1)
✅ Backports have been created
|
@Mergifyio backport 2.6.x |
✅ Backports have been created
|
* Refs #15915: Added test and error trace Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Disabled for intraprocess Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Added new gap requirement check when using separate sending Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Changed the test from a parametrized test to a standalone test Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Added review suggestions Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Fixed Windows warnings Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Added doxygen. Re-added removed assertion Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #15915: Removed unused variables Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
Signed-off-by: Javier Santiago javiersantiago@eprosima.com
Description
When using separate sending, in the event of a change with a sequence number lower than the last one being no longer available in the Writer History a GAP was not being properly sent. This leads to the Reader and Writer being stuck on a cycle of HEARTBEATs and ACKNACKs that request the missing change.
Contributor Checklist
versions.md
file (if applicable).Reviewer Checklist