-
Notifications
You must be signed in to change notification settings - Fork 3
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
Remove Queues and Replace With Disruptor #3
Comments
Strings are used in tests only, and the classes using strings are not part of the core. |
Disruptor makes sense because of this. Have a look at this: https://code.google.com/p/disruptor/wiki/DisruptorWizard Problem with this kind of thing is that queueing locking and switching has over head. This is exactly what I am trying to avoid. See https://code.google.com/p/disruptor/wiki/DisruptorWizard |
Premature optimization is the root of all evil (c) D. Knuth. thanks, On Wed, Sep 4, 2013 at 2:08 AM, Suminda Dharmasena <[email protected]
|
An article "When Disruptor is not a good fit": http://blog.parwy.com/2012/03/when-disruptor-is-not-good-fit.html I don't want to say that I reject using of Disruptor, but want to point out that Disruptor cannot replace classic design and should be considered as option. |
I am looking to see if you can do about 30m transactions per second on a single machine.So the design should be about speed speed and speed alone. http://reactor.github.io/reactor/ these guys have made it upto 100m non network transactions. |
I did not find where they say about 100 millions. LMAX reports of 50 Anyway, I cannot do more than 1 million tps. My goal is not super speed, thanks, On Wed, Sep 4, 2013 at 11:06 PM, Suminda Dharmasena <
|
They say about events/sec, not transactions/sec. Transaction execution thanks, On Thu, Sep 5, 2013 at 12:44 AM, Suminda Dharmasena <
|
Hi,
Lest remove queues / lists and replace with disruptor where possible.
Also try not to use strings for messaging other than the class is for string messaging.
S
The text was updated successfully, but these errors were encountered: