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

Fix DelayedDelivery at the broker level has a default value #9030

Merged
merged 1 commit into from
Dec 23, 2020

Conversation

315157973
Copy link
Contributor

Motivation

delayedDeliveryEnabled at the broker level is enabled by default, but at the namespace level is not set.
But now the value can be obtained directly through admin.namespaces().getDelayedDelivery(namespace).
This makes users very confused, we did not set the policy, but it has value.
Therefore, null should be returned by default

@315157973
Copy link
Contributor Author

/pulsarbot run-failure-checks

@315157973
Copy link
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit 6e74a9b into apache:master Dec 23, 2020
@codelipenghui codelipenghui added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label Jan 7, 2021
codelipenghui pushed a commit that referenced this pull request Jan 7, 2021
### Motivation

`delayedDeliveryEnabled` at the broker level is enabled by default, but at the namespace level is not set.
But now the value can be obtained directly through `admin.namespaces().getDelayedDelivery(namespace)`. 
This makes users very confused, we did not set the policy, but it has value.
Therefore, `null` should be returned by default

(cherry picked from commit 6e74a9b)
@315157973 315157973 deleted the delayed branch February 23, 2021 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked/branch-2.7 Archived: 2.7 is end of life release/2.7.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants