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

Support topic-level inactiveTopicPolicies #7986

Merged
merged 2 commits into from
Sep 5, 2020

Conversation

315157973
Copy link
Contributor

Master Issue: #2688

Motivation

Support topic-level inactiveTopicPolicies

Modifications

Support set/get/remove inactiveTopicPolicies policy on topic level.

Verifying this change

unit tests:

org.apache.pulsar.admin.cli.PulsarAdminToolTest#topics
org.apache.pulsar.broker.service.InactiveTopicDeleteTest#testTopicLevelInActiveTopicApi
org.apache.pulsar.broker.service.InactiveTopicDeleteTest#testTopicLevelInactivePolicyUpdateAndClean
org.apache.pulsar.broker.service.InactiveTopicDeleteTest#testDeleteWhenNoSubscriptionsWithTopicLevelPolicies

@315157973
Copy link
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui added this to the 2.7.0 milestone Sep 5, 2020
@codelipenghui codelipenghui added the doc-required Your PR changes impact docs and you will update later. label Sep 5, 2020
Copy link
Contributor

@codelipenghui codelipenghui left a comment

Choose a reason for hiding this comment

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

👍

@codelipenghui codelipenghui merged commit 4a441db into apache:master Sep 5, 2020
@codelipenghui codelipenghui mentioned this pull request Sep 5, 2020
14 tasks
@315157973 315157973 deleted the inactive branch September 14, 2020 06:45
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Oct 3, 2020
### Motivation
Support topic-level inactiveTopicPolicies

### Modifications
Support set/get/remove inactiveTopicPolicies policy on topic level.
@Anonymitaet Anonymitaet added doc-complete Your PR changes impact docs and the related docs have been already added. and removed doc-required Your PR changes impact docs and you will update later. labels Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-complete Your PR changes impact docs and the related docs have been already added.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants