?
avatar andrepereiradasilva
andrepereiradasilva
11 Dec 2016

Steps to reproduce the issue

Check any travis build. hhvm tests are significally slower than php tests.

At 77% of unit tests it takes a long time (something is strange here).
This is then reflected in hhvm total test time
image

Example: https://travis-ci.org/joomla/joomla-cms

Expected result

hhvm tests takes more or less the same time as php unit tests

Actual result

More than double time.

System information (as much as possible)

N/A

Additional comments

N/A

avatar andrepereiradasilva andrepereiradasilva - open - 11 Dec 2016
avatar joomla-cms-bot joomla-cms-bot - change - 11 Dec 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 11 Dec 2016
avatar andrepereiradasilva andrepereiradasilva - change - 11 Dec 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 11 Dec 2016
avatar photodude
photodude - comment - 11 Dec 2016

There are 2 factors in the total time for the HHVM tests

  1. HHVM is running on the slower sudo required infrastructure not much we can do until after 1Q17 when the Trusty container comes out of beta (the Trusty beta container is currently broken due to a few mysql socket issues)
  2. Some of the tests run slow on HHVM. This could be an issue with the tests, PHPunit, HHVM, or even with Joomla's code.

You can use --debug to see what tests are running at the moment things slow down (unfortunately it's spending about 5 minutes watching the tests then trying to write down which ones are slow)

avatar csthomas
csthomas - comment - 12 Dec 2016

I have prepared a PR #13169.

@photodude If you are interest in speed up unit tests take a look at #11872.

avatar andrepereiradasilva andrepereiradasilva - close - 12 Dec 2016
avatar andrepereiradasilva andrepereiradasilva - change - 12 Dec 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-12-12 16:32:14
Closed_By andrepereiradasilva
avatar andrepereiradasilva andrepereiradasilva - close - 12 Dec 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 12 Dec 2016

closed as we have a PR to test

Add a Comment

Login with GitHub to post a comment