Closed EduPonz closed 1 month ago
@Mergifyio backport 2.13.x 2.10.x 2.6.x
backport 2.13.x 2.10.x 2.6.x
@richiprosima please test this
@richiprosima please test discovery-server
@richiprosima please test discovery-server
@richiprosima please test discovery-server
Description
Backport of #4835 to 2.x
This PR fixes a bug that caused the content filter to also be applied to
unregister
anddisposed
samples. Since in those messages the only fields populated (if any) are the ones annotated with@key
, theunregister
anddispose
samples did not pass the filter (in general) and thus were being discarded. This caused several issues:unregister
ordispose
followed by awrite
were triggeringsample_lost
events, as the received sequence numbers were not consecutive (because of the filtering out of theunregister
/dispose
).This PR fixes these issues by only querying for sample relevance when the
CacheChange
kind isALIVE
.@Mergifyio backport 2.13.x 2.10.x 2.6.x
Contributor Checklist
versions.md
file (if applicable).Reviewer Checklist