Skip to content

[#24288] Clarify default empty Partition behavior in Partition QoS#1237

Open
Danipiza wants to merge 1 commit intomasterfrom
hotfix/empty-partition
Open

[#24288] Clarify default empty Partition behavior in Partition QoS#1237
Danipiza wants to merge 1 commit intomasterfrom
hotfix/empty-partition

Conversation

@Danipiza
Copy link
Copy Markdown
Contributor

@Danipiza Danipiza commented Mar 24, 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.

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.

@Danipiza Danipiza force-pushed the hotfix/empty-partition branch from 062805c to 76c5348 Compare March 24, 2026 15:15
@zesk1999 zesk1999 self-requested a review March 25, 2026 12:47
@Danipiza Danipiza requested a review from zesk1999 March 25, 2026 14:53
Copy link
Copy Markdown
Contributor

@zesk1999 zesk1999 left a comment

Choose a reason for hiding this comment

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

LGTM with green CI

@Danipiza Danipiza added this to the v3.6.0 milestone Mar 26, 2026
Signed-off-by: danipiza <dpizarrogallego@gmail.com>
@Danipiza Danipiza force-pushed the hotfix/empty-partition branch from 76c5348 to b88ea11 Compare March 26, 2026 08:07
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