Skip to content

[24094] Change resource limits default values#1188

Merged
MiguelCompany merged 4 commits intomasterfrom
feature/qos_consistencies
Jan 22, 2026
Merged

[24094] Change resource limits default values#1188
MiguelCompany merged 4 commits intomasterfrom
feature/qos_consistencies

Conversation

@MiguelCompany
Copy link
Member

@MiguelCompany MiguelCompany commented Jan 21, 2026

Description

This changes the documentation regrarding the default values of max_samples, max_instances, and max_samples_per_instance.

It also clarifies the behavior on topics without keys, and that allocated_samples is limited by max_samples

It would be nice to backport commits d70aaca and 8792629

Related implementation PR:

Contributor Checklist

  • Commit messages follow the project guidelines.
  • Code snippets related to the added documentation have been provided.
  • Documentation tests pass locally.
  • N/A: The Pro version badge has been added if the documented feature is exclusive to Fast DDS Pro.
  • 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.

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
Copy link
Contributor

@raulojeda22 raulojeda22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MiguelCompany MiguelCompany merged commit 02f5bd8 into master Jan 22, 2026
4 checks passed
@MiguelCompany MiguelCompany deleted the feature/qos_consistencies branch January 22, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants