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

Add benchmark for spray #278

Merged
merged 1 commit into from
May 13, 2013
Merged

Add benchmark for spray #278

merged 1 commit into from
May 13, 2013

Conversation

jrudolph
Copy link
Contributor

Currently, only the json benchmark is supported.

@jrudolph
Copy link
Contributor Author

If merged, this would fix #80.

@bhauer
Copy link
Contributor

bhauer commented May 13, 2013

Thanks for the contribution, @jrudolph!

@jrudolph
Copy link
Contributor Author

We just found two or three things we'll have to update. @sirthias will also give his LGTM when we are finished.

@sirthias
Copy link

Ok, this should now provide a nice approximation of how spray would currently perform for the JSON benchmark.
We'll see whether we can add the required bits for the other tests as well later, but for round 5 this is all we can currently supply.
We are quite curious as to how spray will fare, so it'd be great if you could get this PR into the suite before round 5 is started.

Please let us know if you need any help with setting the right attributes for spray! (We couldn't find the data origin for these values (classification, etc.) in the repo...)

@pfalls1 pfalls1 merged commit 2160863 into TechEmpower:master May 13, 2013
@pfalls1
Copy link
Contributor

pfalls1 commented May 13, 2013

Thanks @jrudolph and @sirthias. This looks good.

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

Successfully merging this pull request may close these issues.

None yet

4 participants