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 maxProducersPerTopic cannot be disabled at the namespace level #9157

Merged
merged 1 commit into from
Jan 11, 2021

Conversation

315157973
Copy link
Contributor

Master Issue: #9146

Motivation

maxProducersPerTopic cannot be disabled at the namespace-level

Modifications

Let maxProducersPerTopic can be null and no longer uses broker-level policy as the default value

Verifying this change

AdminApiTest2#testMaxProducersPerTopicUnlimited

@315157973
Copy link
Contributor Author

/pulsarbot run-failure-checks

1 similar comment
@315157973
Copy link
Contributor Author

/pulsarbot run-failure-checks

@sijie sijie merged commit ab8802b into apache:master Jan 11, 2021
codelipenghui pushed a commit that referenced this pull request Jan 14, 2021
…el (#9157)

Master Issue: #9146

`maxProducersPerTopic` cannot be disabled at the namespace-level

Let `maxProducersPerTopic` can be null and no longer uses broker-level policy as the default value

AdminApiTest2#testMaxProducersPerTopicUnlimited

(cherry picked from commit ab8802b)
@codelipenghui codelipenghui added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label Jan 14, 2021
freeznet pushed a commit to streamnative/pulsar-archived that referenced this pull request Jan 14, 2021
…el (apache#9157)

Master Issue: apache#9146

`maxProducersPerTopic` cannot be disabled at the namespace-level

Let `maxProducersPerTopic` can be null and no longer uses broker-level policy as the default value

AdminApiTest2#testMaxProducersPerTopicUnlimited

(cherry picked from commit ab8802b)
@315157973 315157973 deleted the max_producers_per_topic branch March 12, 2021 13:52
merlimat pushed a commit to merlimat/pulsar that referenced this pull request Apr 6, 2021
…el (apache#9157)

Master Issue: apache#9146

### Motivation
`maxProducersPerTopic` cannot be disabled at the namespace-level

### Modifications
Let `maxProducersPerTopic` can be null and no longer uses broker-level policy as the default value

### Verifying this change
AdminApiTest2#testMaxProducersPerTopicUnlimited
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

3 participants