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

adds benchmark test to validate padding behaviour #19

Open
jinCN opened this issue May 30, 2014 · 3 comments
Open

adds benchmark test to validate padding behaviour #19

jinCN opened this issue May 30, 2014 · 3 comments

Comments

@jinCN
Copy link

jinCN commented May 30, 2014

with a long type value,the implimentation of java Sequence will pad 7 bytes before the value and pad 7 bytes after the value,but why did you only pad 7 bytes after the value?

@fsaintjacques
Copy link
Owner

I'm trying to find the old source for the java disruptor. Also note that they're padding longs, not bytes.

@fsaintjacques
Copy link
Owner

@fsaintjacques fsaintjacques changed the title padding 7 bytes? adds benchmark test to validate padding behaviour Jul 6, 2015
@fsaintjacques
Copy link
Owner

There's also an interesting point regarding this:

http://stackoverflow.com/questions/29199779/false-sharing-and-128-byte-alignment-padding

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

2 participants