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
ARTEMIS-2292 - Fix queue level auto-delete for topic subscription queue #213
ARTEMIS-2292 - Fix queue level auto-delete for topic subscription queue #213
Conversation
(cherry picked from commit c4fce10) NO-DOWNSTREAM-JIRA: This is a fix on a new functionality, not yet released on the product branch.
@andytaylor this is important to avoid future compatibility issues on the product branch. Please merge it before tagging it. |
Additional bits missed (cherry picked from commit 0df9205) NO-DOWNSTREAM-JIRA: This is a fix on a new functionality, not yet released on the product branch.
@@ -3160,6 +3160,14 @@ | |||
</xsd:annotation> | |||
</xsd:element> | |||
|
|||
<xsd:element name="auto-delete-created-queues" type="xsd:boolean" default="true" maxOccurs="1" minOccurs="0"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't here be default="false" @clebertsuconic ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected anyway. I am just curious if this is not a mistake in schema.
All tests are passing, manually played with it a little bit, seems to be working as expected. Have only 1 comment above. Else giving it a green light. |
Adjust auto-delete defaults based on if queue autocreated (cherry picked from commit 369c429) NO-DOWNSTREAM-JIRA: This is a fix on a new functionality, not yet released on the product branch.
(cherry picked from commit 821f291) NO-DOWNSTREAM-JIRA: This is a fix on a new functionality, not yet released on the product branch.
merged |
(cherry picked from commit c4fce10)
NO-DOWNSTREAM-JIRA: This is a fix on a new functionality, not yet released on the product branch.