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

HORNETQ-1417 OpenWire support -- Producer Flow Control #1870

Merged
merged 1 commit into from Sep 29, 2014

Conversation

gaohoward
Copy link
Member

Implement OpenWire producer flow control. Based on configuration, when
a queue's memory space is full, the producer can either be blocked (via
client side window size or server side blocking), or get a relevant
failure exception. The details are given in this link:

http://activemq.apache.org/producer-flow-control.html

@gaohoward
Copy link
Member Author

close for now.

@gaohoward gaohoward closed this Sep 29, 2014
@clebertsuconic
Copy link
Member

We are not closing PRs any longer.. that causes a lot of confusion on github

@clebertsuconic
Copy link
Member

github has been creating issues with closing a PR and rebasing it... just let us know when you're ready.. nobody should merge your PR

We are closing a PR now only when it's meant for good

@gaohoward
Copy link
Member Author

@celbert actually the join breaks the tests. It shouldn't join() at the end of it. It is supposed to go on doing other stuff while the thread is dying. I'll find a proper place to join it.

@clebertsuconic
Copy link
Member

teardown?

@gaohoward
Copy link
Member Author

I'll give it a try.

Implement OpenWire producer flow control. Based on configuration, when
a queue's memory space is full, the producer can either be blocked (via
client side window size or server side blocking), or get a relevant
failure exception. The details are given in this link:

http://activemq.apache.org/producer-flow-control.html
@gaohoward
Copy link
Member Author

yes it works. I've done a forced push just now.

clebertsuconic added a commit that referenced this pull request Sep 29, 2014
HORNETQ-1417 OpenWire support -- Producer Flow Control
@clebertsuconic clebertsuconic merged commit d325f90 into hornetq:master Sep 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants