Skip to content

Conversation

@rdhabalia
Copy link
Contributor

Motivation

It addresses #3985 by introducing publish rate-limiting on the topic.

@rdhabalia rdhabalia added this to the 2.4.0 milestone Apr 5, 2019
@rdhabalia rdhabalia self-assigned this Apr 5, 2019
@rdhabalia
Copy link
Contributor Author

rerun java8 tests
rerun integration tests

@sijie
Copy link
Member

sijie commented Jun 9, 2019

do we need this for 2.4.0 or shall we move it to 2.5.0?

@sijie sijie modified the milestones: 2.4.0, 2.5.0 Jun 14, 2019
@sijie
Copy link
Member

sijie commented Jun 14, 2019

@rdhabalia I moved this feature to 2.5.0. If it is needed in 2.4.0, please move it back :-)

@rdhabalia rdhabalia force-pushed the pub_throttle branch 2 times, most recently from dba94a8 to 5368cea Compare July 3, 2019 21:22
@rdhabalia
Copy link
Contributor Author

rerun java8 tests

@rdhabalia
Copy link
Contributor Author

@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
@rdhabalia
Copy link
Contributor Author

rerun java8 tests

2 similar comments
@rdhabalia
Copy link
Contributor Author

rerun java8 tests

@rdhabalia
Copy link
Contributor Author

rerun java8 tests

@rdhabalia
Copy link
Contributor Author

rerun java8 tests
rerun integration tests

@rdhabalia rdhabalia merged commit 7686542 into apache:master Oct 25, 2019
@wolfstudy wolfstudy modified the milestones: 2.5.0, 2.4.2 Nov 19, 2019
@rdhabalia rdhabalia deleted the pub_throttle branch November 19, 2019 19:12
shiv4289 added a commit to shiv4289/pulsar that referenced this pull request May 6, 2020
It seems apache#3986 was not documented. Adding documentation for the namespace cli commands.
sijie pushed a commit that referenced this pull request May 7, 2020
It seems #3986 was not documented. Adding documentation for the namespace cli commands.
Huanli-Meng pushed a commit to Huanli-Meng/pulsar that referenced this pull request May 27, 2020
It seems apache#3986 was not documented. Adding documentation for the namespace cli commands.
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
It seems apache#3986 was not documented. Adding documentation for the namespace cli commands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants