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 subscribe rate on topic level #7991

Merged

Conversation

hangc0276
Copy link
Contributor

Modifications

Support set subscribe rate on topic level.
Support get subscribe rate on topic level.
Support remove subscribe rate on topic level.

Verifying this change

This change added tests and can be verified as follows:

  • test set topic subscribe rate
  • test remove topic subscribe rate
  • test get topic topic subscribe rate
  • test disabled topic subscribe rate

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (don't know)
  • The default values of configurations: (yes / no)
  • The wire protocol: (yes / no)
  • The rest endpoints: (yes / no)
  • The admin cli options: (yes)
  • Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs / JavaDocs)

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

1 similar comment
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@jiazhai
Copy link
Member

jiazhai commented Sep 11, 2020

@hangc0276 Would you please help update with latest master?

@Jennifer88huang-zz
Copy link
Contributor

@hangc0276 thanks for your contribution. There are some conflicting files in this PR, could you please resolve those conflicts? thank you.

@hangc0276
Copy link
Contributor Author

@hangc0276 thanks for your contribution. There are some conflicting files in this PR, could you please resolve those conflicts? thank you.

Thanks for your feedback @jennifer88huang @jiazhai . The code has a bug and i am adding more tests and try to fix the bug. I will commit the code soon.

@Jennifer88huang-zz
Copy link
Contributor

@hangc0276 thank you.

@hangc0276 hangc0276 force-pushed the add_subscribe_rate_on_topic_level branch from f1c9545 to 0e623ea Compare September 14, 2020 15:36
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

5 similar comments
@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
Copy link
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit afe6413 into apache:master Sep 21, 2020
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Oct 3, 2020
Support set subscribe rate on topic level.
Support get subscribe rate on topic level.
Support remove subscribe rate on topic level.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants