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

Optionally restrict the max number of subscribers #32

Open
brianoliver opened this issue Mar 9, 2013 · 10 comments
Open

Optionally restrict the max number of subscribers #32

brianoliver opened this issue Mar 9, 2013 · 10 comments

Comments

@brianoliver
Copy link
Contributor

It can sometimes be useful to restrict the amount of subscribers in a Queue destination. An example of that would be to create multiple queues with a single subscriber in each one, and hash an object to each one (something which can be a weak guarantee of in-order processing for multiple version of the same object).

@brianoliver
Copy link
Contributor Author

Reported by ioannism

@brianoliver
Copy link
Contributor Author

@ioannisM said:
I already have a patch in place for this, is it possible to attach it to the ticket?

@brianoliver
Copy link
Contributor Author

@brianoliver said:
If you have executed an Oracle Contributors Agreement, then we can accept the patch.

If you're planning on doing major development work, the best way to submit patches is to fork the Incubator into your own repository, make the changes there, re-base your fork and then create a Pull Request. We can then review the submission together before it they are integrated directly into the development branch.

@brianoliver
Copy link
Contributor Author

@ioannisM said:
Hi Brian, thanks. I have signed an OCA for OpenJDK only though, I'll submit one tomorrow morning for Incubator. Insofar as major dev work, who knows I'd be more than happy to contribute though.

@brianoliver
Copy link
Contributor Author

@brianoliver said:
Great. There's no need to sign it again for the Incubator. Once I can confirm you're on the OCA list
(which is here: http://www.oracle.com/technetwork/community/oca-486395.html), then we can accept patches.

If you're not on this list then you'll need to submit a new OCA or follow up to why you're not yet listed

@brianoliver
Copy link
Contributor Author

@ioannisM said:
Submitted the OCA very recently (Friday 8th) so it's still making its way through the system I'll post a comment here once I get notification , cheers

@brianoliver
Copy link
Contributor Author

@ioannisM said:
The OCA has been accepted and I'm now on the list

@brianoliver
Copy link
Contributor Author

@brianoliver said:
Awesome. I updated your involvement to the level of "developer". Please feel free to attach source here, or fork + create a pull request on the project.

@brianoliver
Copy link
Contributor Author

@ioannisM said:
Already forked and submitted a pull request or I could submit a patch if you want, whatever is more convenient.

@brianoliver
Copy link
Contributor Author

This issue was imported from JIRA COHINC-32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant