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

Allow for topic deletions with regex consumers #5230

Merged
merged 5 commits into from
Sep 27, 2019

Conversation

merlimat
Copy link
Contributor

Motivation

Currently it's not possible to delete topics when there is a regex consumer attached to them. The reason is that the regex consumer will immediately reconnect and cause the topic to be re-created.

Modifications

  • Added a flag in CommandSubscribe so that a regex consumer will never trigger the creation of a topic.
  • Subscribing to a non-existing topic, will give a specific error, that the consumer will interpret as a permanent failure and thus will stop retrying.

@merlimat merlimat added the type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages label Sep 19, 2019
@merlimat merlimat added this to the 2.4.2 milestone Sep 19, 2019
@merlimat merlimat self-assigned this Sep 19, 2019
@merlimat
Copy link
Contributor Author

run cpp tests
run java8 tests

@merlimat
Copy link
Contributor Author

run integration tests

@merlimat
Copy link
Contributor Author

run java8 tests
run integration tests

@sijie
Copy link
Member

sijie commented Sep 22, 2019

@jiazhai since you are familiar with regex subscription and also involved in the original discussion, can you please take a look at this change?

Copy link
Contributor

@ivankelly ivankelly left a comment

Choose a reason for hiding this comment

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

Logic looks good.

@merlimat
Copy link
Contributor Author

run java8 tests

1 similar comment
@merlimat
Copy link
Contributor Author

run java8 tests

@merlimat merlimat merged commit 07bb845 into apache:master Sep 27, 2019
@wolfstudy
Copy link
Member

The pr change the proto file, will move the milestone to 2.5.0

@EugenDueck
Copy link
Contributor

I've just created #6136 to fix the release notes

sijie pushed a commit that referenced this pull request Jan 25, 2020
### Motivation

*A change that had to be moved to 2.5.0 (because it [changed the .proto file](#5230 (comment))), is still listed as having been done in 2.4.2 in the release notes.*

### Modifications

*Move the change from 2.4.2 to 2.5.0.*
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
### Motivation

*A change that had to be moved to 2.5.0 (because it [changed the .proto file](apache#5230 (comment))), is still listed as having been done in 2.4.2 in the release notes.*

### Modifications

*Move the change from 2.4.2 to 2.5.0.*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants