Skip to content

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

Open
mergify[bot] wants to merge 1 commit into3.4.xfrom
mergify/bp/3.4.x/pr-1237
Open

[#24288] Clarify default empty Partition behavior in Partition QoS (backport #1237)#1245
mergify[bot] wants to merge 1 commit into3.4.xfrom
mergify/bp/3.4.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 6007

@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).

@Danipiza Danipiza force-pushed the mergify/bp/3.4.x/pr-1237 branch from 5137d1d to 0dc604d Compare March 30, 2026 12:56
@Danipiza Danipiza requested a review from zesk1999 March 30, 2026 12:57
@Danipiza Danipiza requested review from Danipiza and removed request for zesk1999 March 31, 2026 08:53
Signed-off-by: danipiza <dpizarrogallego@gmail.com>
@Danipiza Danipiza force-pushed the mergify/bp/3.4.x/pr-1237 branch from 0dc604d to e7d10cd Compare March 31, 2026 08:55
@Danipiza Danipiza requested review from Danipiza and removed request for Danipiza March 31, 2026 08: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

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