-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[pulsar-broker] Introduce publish rate-limiting on topic #3986
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
Conversation
|
rerun java8 tests |
|
do we need this for 2.4.0 or shall we move it to 2.5.0? |
|
@rdhabalia I moved this feature to 2.5.0. If it is needed in 2.4.0, please move it back :-) |
dba94a8 to
5368cea
Compare
|
rerun java8 tests |
pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/Namespaces.java
Outdated
Show resolved
Hide resolved
|
@merlimat can you also review this PR before we merge? we have need of this feature so, want to merge if possible. |
disable auto-read on cnx for publish throttling Clean up
2124500 to
7400037
Compare
7400037 to
20484a3
Compare
|
rerun java8 tests |
2 similar comments
|
rerun java8 tests |
|
rerun java8 tests |
|
rerun java8 tests |
It seems apache#3986 was not documented. Adding documentation for the namespace cli commands.
It seems #3986 was not documented. Adding documentation for the namespace cli commands.
It seems apache#3986 was not documented. Adding documentation for the namespace cli commands.
It seems apache#3986 was not documented. Adding documentation for the namespace cli commands.
Motivation
It addresses #3985 by introducing publish rate-limiting on the topic.