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

Performance regression #17

Open
whatsthecraic opened this issue May 10, 2020 · 2 comments
Open

Performance regression #17

whatsthecraic opened this issue May 10, 2020 · 2 comments

Comments

@whatsthecraic
Copy link

whatsthecraic commented May 10, 2020

Hi there,
as I've repeated another round of our internal experiments, I noticed a regression compared to the results I obtained back in February. For instance I am accounting a difference of ~2.9x for insertions on graph500 with s.f. 24 on a relatively modern dual socket Xeon, with 20 cores in total. My current branch tracks 623d3f8 after issue #16, while the results from February are likely based on the master branch that was available at the end of December'19, after issue #11 , but unfortunately I didn`t record the precise commit that was used at the time.
The throughput obtained is the same as before with up to two threads to execute the insertions. After that, the scalability doesn't improve any more in my current measurements, while in February I obtained the best results with 12 threads.

I wonder what is the reason for this difference and if there is something we can do to improve the current state. The patch for #16 only alters the creation of static views, which are not even used in this experiment.

Thanks,
Dean

@pradeep-k
Copy link
Member

Hi Dean,
I am not sure why performance regression is failing. I can take a look into it by month end only.

If you have any functionality bug, let me know. Otherwise I will check the performance issue by month end.

@whatsthecraic
Copy link
Author

Hey there,

thank you for the quick reply. It doesn't affect any functionality. I'll wait for a future update then.

Thanks,
Dean

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

No branches or pull requests

2 participants