Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[20503] Clarify history depth vs max_samples_per_instance inconsistency #679

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

EduPonz
Copy link

@EduPonz EduPonz commented Feb 21, 2024

@JesusPoderoso
Copy link
Contributor

@richiprosima please test docs

@JesusPoderoso JesusPoderoso added this to the v2.13.3 milestone Feb 22, 2024
@EduPonz EduPonz force-pushed the hotfix/default-unlimited-resources branch from 912b895 to 6091ef4 Compare February 22, 2024 10:36
@EduPonz EduPonz changed the title [20503] Change default resource limits to infinite [20503] Clarify history depth vs max_samples_per_instance inconsistency Feb 22, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
@EduPonz EduPonz force-pushed the hotfix/default-unlimited-resources branch from 6091ef4 to a66f8f6 Compare February 22, 2024 10:39
Copy link
Contributor

@JesusPoderoso JesusPoderoso left a comment

Choose a reason for hiding this comment

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

LGTM

@EduPonz
Copy link
Author

EduPonz commented Feb 22, 2024

@richiprosima please test this

@EduPonz EduPonz merged commit f5ab6a9 into master Feb 22, 2024
4 checks passed
@EduPonz EduPonz deleted the hotfix/default-unlimited-resources branch February 22, 2024 13:49
@JesusPoderoso
Copy link
Contributor

@Mergifyio backport 2.10.x

Copy link

mergify bot commented Feb 22, 2024

backport 2.10.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Feb 22, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit f5ab6a9)
JesusPoderoso pushed a commit that referenced this pull request Feb 22, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
JesusPoderoso pushed a commit that referenced this pull request Feb 22, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
elianalf pushed a commit that referenced this pull request Mar 13, 2024
) (#674)

* Improve History QoS documentation and compatibility rules (#664)

* Refs #20401: Improve History documentation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Update example to out last standard tabs format

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Update all file examples

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Add NIT to explanation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Fix doc tab

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply spaces rev suggestion

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply documentation rev suggestions

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply documentation update based on internal discussion

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Remove old explanation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Improve documentation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply NIT

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
(cherry picked from commit 157375d)

* Clarify depth vs max_samples_per_instance inconsistency (#679)

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

---------

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
Co-authored-by: Jesús Poderoso <120394830+JesusPoderoso@users.noreply.github.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
MiguelCompany pushed a commit that referenced this pull request Mar 14, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit f5ab6a9)

Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
Mario-DL pushed a commit that referenced this pull request Apr 16, 2024
* Improve History QoS documentation and compatibility rules (#664)

* Refs #20401: Improve History documentation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Update example to out last standard tabs format

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Update all file examples

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Add NIT to explanation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Fix doc tab

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply spaces rev suggestion

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply documentation rev suggestions

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply documentation update based on internal discussion

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Remove old explanation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Improve documentation

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Refs #20401: Apply NIT

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
(cherry picked from commit 157375d)

# Conflicts:
#	docs/fastdds/dds_layer/core/policy/standardQosPolicies.rst

* Refs #20401: Fix conlficts

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>

* Clarify depth vs max_samples_per_instance inconsistency (#679)

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

---------

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
Co-authored-by: Jesús Poderoso <120394830+JesusPoderoso@users.noreply.github.com>
Co-authored-by: JesusPoderoso <jesuspoderoso@eprosima.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
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