Skip to content

Conversation

suutari-ai
Copy link
Member

It seems that Jest runs much faster in Travis when it's ran with this
flag. E.g. for one particular project, this made the frontend testing
time drop from 20 minutes to 2 minutes.

It seems that Jest runs much faster in Travis when it's ran with this
flag.  E.g. for one particular project, this made the frontend testing
time drop from 20 minutes to 2 minutes.
@suutari-ai suutari-ai requested review from ehaivala and RauliL January 15, 2018 08:55
Copy link

@ehaivala ehaivala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright

@ehaivala ehaivala merged commit 8765255 into andersinno:master Jan 15, 2018
@frwickst
Copy link

I know this is merged, but would -w have been enough here?

https://facebook.github.io/jest/docs/en/cli.html#maxworkers-num

@suutari-ai
Copy link
Member Author

I know this is merged, but would -w have been enough here?

You mean that so it would have used the number of CPUs instead of hard coded 4? Might have been good too, but the number of CPUs is irrelevant here, since the constraining resource here is amount of memory rather than amount of CPUs. Therefore some hard coded small value might be better. (E.g. if they make e.g. 64 CPUs available for the job runner container in Travis, but it still has very small memory.)

@suutari-ai suutari-ai deleted the speedup-jest-in-travis branch January 15, 2018 09:20
@frwickst
Copy link

That is indeed true 🙂 Just wanted to check 👍

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.

3 participants