-
Notifications
You must be signed in to change notification settings - Fork 103
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
TechEmpower Benchmarks #23
Comments
Thanks for pointing this out! |
Ongoing in https://github.com/circlespainter/FrameworkBenchmarks ( |
I'd say it's ready for PRs but I'd like to understand better a few more things about the Framework Benchmarks; no feedback so far via https://groups.google.com/forum/#!topic/framework-benchmarks/Gujdipuua7Q though. @synapsechris maybe you can help answering those questions? |
@circlespainter nice work on the framework and it's great to see you've added tests cases into TechEmpower. I've been trying to do some testing locally and noticed that the comsat version needs to be upgraded to |
@yunspace Thanks, I was planning to update the PRs to 0.5.0 stable once released so I'll do that shortly. |
hi @circlespainter I tried to run the tests again using your latest commits:
I think for
Otherwise there'll be compilation errors. Afterward fixing that I got the following error:
Which seem to be caused by the default mvn being 3.0.5 and not the 3.1.0 required by capsule-maven-plugin: chrisdchristo/capsule-maven-plugin#4 Were you able run these tests locally in |
btw if you haven't tried running the benchmark locally, the steps I've been following steps are:
as per documentation here http://frameworkbenchmarks.readthedocs.org/en/latest/Development/Installation-Guide/#vagrant-development-environment |
Thanks for looking into it; yes bringing it up-to-date took more than I expected and indeed I'll need to re-run a test in the Vagrant environment as well. I'll get this sorted out by the end of the week. |
@yunspace Ok, now both run fine in my Vagrant env but they fail on CI; it looks like the
Is there anything in the framework's build source that could cause this (apart from the dependency to java8 - which is there only because it's more performant, although the framework support java7 too)? |
@yunspace Just tried with the most recent |
I had some issues updating to On Mon, Sep 21, 2015 at 1:20 AM, Fabio Tudone notifications@github.com
|
The servlet framework has been merged, still having a strange failure on CI on the webactors one (on a fresh |
The web-only tests of |
https://www.techempower.com/benchmarks/
The text was updated successfully, but these errors were encountered: