[20849] New `max_message_size` property to limit output datagrams size #770

elianalf · 2024-05-14T08:57:02Z

Description

This PR adds the documentation for max_message_size property in RTPS Writer and RTPS Participant.

@Mergifyio backport 2.14.x 2.13.x 2.10.x 2.6.x

Related implementation PR:

[20849] New max_message_size property to limit output datagrams size Fast-DDS#4777

Contributor Checklist

Commit messages follow the project guidelines.
Code snippets related to the added documentation have been provided.
Documentation tests pass locally.
Applicable backports have been included in the description.

Reviewer Checklist

The PR has a milestone assigned.
The title and description correctly express the PR's purpose.
Check contributor checklist is correct.
CI passes without warnings or errors.

code/DDSCodeTester.cpp

code/XMLTester.xml

docs/fastdds/property_policies/non_consolidated_qos.rst

code/XMLTester.xml

docs/fastdds/property_policies/non_consolidated_qos.rst

Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com>

MiguelCompany · 2024-05-17T09:35:55Z

@richiprosima Please test_3 docs

MiguelCompany · 2024-05-17T15:22:05Z

@Mergifyio backport 2.14.x 2.13.x 2.10.x 2.6.x

mergify · 2024-05-17T15:22:12Z

backport 2.14.x 2.13.x 2.10.x 2.6.x

✅ Backports have been created

#774 [20849] New max_message_size property to limit output datagrams size (backport #770) has been created for branch 2.14.x but encountered conflicts
#775 [20849] New max_message_size property to limit output datagrams size (backport #770) has been created for branch 2.13.x but encountered conflicts
#776 [20849] New max_message_size property to limit output datagrams size (backport #770) has been created for branch 2.10.x but encountered conflicts
#777 [20849] New max_message_size property to limit output datagrams size (backport #770) has been created for branch 2.6.x but encountered conflicts

* Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp # docs/fastdds/property_policies/non_consolidated_qos.rst

* Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp

* Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp # docs/fastdds/property_policies/non_consolidated_qos.rst

* Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp # code/XMLTester.xml # docs/fastdds/property_policies/non_consolidated_qos.rst

* Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp

* Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp # docs/fastdds/property_policies/non_consolidated_qos.rst

* Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8)

…#776) * Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp # docs/fastdds/property_policies/non_consolidated_qos.rst Co-authored-by: elianalf <62831776+elianalf@users.noreply.github.com>

…#774) * Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp Co-authored-by: elianalf <62831776+elianalf@users.noreply.github.com>

…#775) * Refs #20849: Add max_message_size property policy docs Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs 20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> * Refs #20849: Apply suggestions Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> --------- Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com> (cherry picked from commit 91d96b8) # Conflicts: # code/DDSCodeTester.cpp # docs/fastdds/property_policies/non_consolidated_qos.rst Co-authored-by: elianalf <62831776+elianalf@users.noreply.github.com>

elianalf added this to the v3.0.0 milestone May 14, 2024

elianalf changed the title ~~Parse max_message_size property in RTPS Writer and RTPS Participant~~ [20489] Parse max_message_size property in RTPS Writer and RTPS Participant May 14, 2024

elianalf mentioned this pull request May 14, 2024

[20849] New max_message_size property to limit output datagrams size eProsima/Fast-DDS#4777

Merged

15 tasks

elianalf changed the title ~~[20489] Parse max_message_size property in RTPS Writer and RTPS Participant~~ [20849] Parse max_message_size property in RTPS Writer and RTPS Participant May 14, 2024

elianalf force-pushed the feature/20489/master branch from 5d31122 to 485fcb3 Compare May 16, 2024 07:24

MiguelCompany requested changes May 16, 2024

View reviewed changes

code/XMLTester.xml Outdated Show resolved Hide resolved

docs/fastdds/property_policies/non_consolidated_qos.rst Outdated Show resolved Hide resolved

docs/fastdds/property_policies/non_consolidated_qos.rst Outdated Show resolved Hide resolved

MiguelCompany requested changes May 16, 2024

View reviewed changes

docs/fastdds/property_policies/non_consolidated_qos.rst Show resolved Hide resolved

docs/fastdds/property_policies/non_consolidated_qos.rst Show resolved Hide resolved

MiguelCompany changed the title ~~[20849] Parse max_message_size property in RTPS Writer and RTPS Participant~~ [20849] New max_message_size property to limit output datagrams size May 16, 2024

elianalf added 3 commits May 17, 2024 11:25

Refs #20849: Add max_message_size property policy docs

2c217af

Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com>

Refs 20849: Apply suggestions

fd0303c

Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com>

Refs #20849: Apply suggestions

8e81d33

Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com>

elianalf force-pushed the feature/20489/master branch from 6a678f9 to 8e81d33 Compare May 17, 2024 09:26

MiguelCompany approved these changes May 17, 2024

View reviewed changes

MiguelCompany added the ci-pending label May 17, 2024

elianalf added ready-to-merge and removed ci-pending labels May 17, 2024

mergify bot mentioned this pull request May 17, 2024

[20849] New max_message_size property to limit output datagrams size (backport #4777) eProsima/Fast-DDS#4805

Merged

15 tasks

MiguelCompany merged commit 91d96b8 into master May 17, 2024
4 checks passed

MiguelCompany deleted the feature/20489/master branch May 17, 2024 15:21

This was referenced May 17, 2024

[20849] New max_message_size property to limit output datagrams size (backport #770) #774

Merged

[20849] New max_message_size property to limit output datagrams size (backport #770) #775

Merged

mergify bot mentioned this pull request May 17, 2024

[20849] New max_message_size property to limit output datagrams size (backport #770) #776

Merged

8 tasks

mergify bot mentioned this pull request Jun 5, 2024

[20849] New max_message_size property to limit output datagrams size (backport #4777) (backport #4807) eProsima/Fast-DDS#4899

Merged

15 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[20849] New `max_message_size` property to limit output datagrams size #770

[20849] New `max_message_size` property to limit output datagrams size #770

elianalf commented May 14, 2024 •

edited by MiguelCompany

Loading

MiguelCompany commented May 17, 2024

MiguelCompany commented May 17, 2024

mergify bot commented May 17, 2024 •

edited

Loading

[20849] New max_message_size property to limit output datagrams size #770

[20849] New max_message_size property to limit output datagrams size #770

Conversation

elianalf commented May 14, 2024 • edited by MiguelCompany Loading

Description

Contributor Checklist

Reviewer Checklist

MiguelCompany commented May 17, 2024

MiguelCompany commented May 17, 2024

mergify bot commented May 17, 2024 • edited Loading

✅ Backports have been created

[20849] New `max_message_size` property to limit output datagrams size #770

[20849] New `max_message_size` property to limit output datagrams size #770

elianalf commented May 14, 2024 •

edited by MiguelCompany

Loading

mergify bot commented May 17, 2024 •

edited

Loading