Skip to content

[#24288] Clarify default empty Partition behavior in Partition QoS (backport #1237)#1243

Merged
Danipiza merged 1 commit into2.14.xfrom
mergify/bp/2.14.x/pr-1237
Mar 31, 2026
Merged

[#24288] Clarify default empty Partition behavior in Partition QoS (backport #1237)#1243
Danipiza merged 1 commit into2.14.xfrom
mergify/bp/2.14.x/pr-1237

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify bot commented Mar 30, 2026

Description

This PR clarifies how the default partition works in DDS Partition QoS.

Changes:

  • Explicitly states that the default partition name is the empty string ("").
  • Clarifies that {} is equivalent to {""}.
  • Keeps tables aligned with the empty/default partition semantics.

PR FastDDS 6008

@Mergifyio backport 2.14.x 3.2.x 3.4.x 3.5.x

Contributor Checklist

  • Commit messages follow the project guidelines.
  • Code snippets related to the added documentation have been provided.
  • Documentation tests pass locally.
  • 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.

This is an automatic backport of pull request #1237 done by [Mergify](https://mergify.com).

Signed-off-by: danipiza <dpizarrogallego@gmail.com>
@Danipiza Danipiza force-pushed the mergify/bp/2.14.x/pr-1237 branch from cd2a7ba to ff861d5 Compare March 30, 2026 12:54
@Danipiza Danipiza requested review from Danipiza and zesk1999 and removed request for zesk1999 March 30, 2026 12:56
Copy link
Copy Markdown
Contributor

@Danipiza Danipiza left a comment

Choose a reason for hiding this comment

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

LGTM

@Danipiza Danipiza merged commit b7ffb29 into 2.14.x Mar 31, 2026
5 checks passed
@Danipiza Danipiza deleted the mergify/bp/2.14.x/pr-1237 branch March 31, 2026 09:28
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.

1 participant