Parallelize longest tests #328

Merged
fyrchik merged 7 commits from fyrchik/frostfs-node:parallelize-tests into master 2023-05-12 09:45:05 +00:00
Owner

Related #321
TODO: metabase

Related #321 TODO: metabase
fyrchik force-pushed parallelize-tests from 153ad4f020 to 445094594c 2023-05-05 13:48:23 +00:00 Compare
fyrchik force-pushed parallelize-tests from 445094594c to 1e2ca79829 2023-05-05 14:06:11 +00:00 Compare
fyrchik changed title from WIP: Parallelize longest tests to Parallelize longest tests 2023-05-06 05:15:11 +00:00
fyrchik requested review from storage-core-committers 2023-05-10 14:27:56 +00:00
fyrchik requested review from storage-core-developers 2023-05-10 14:27:56 +00:00

Maybe add make test duration measurements before/after?

Maybe add `make test` duration measurements before/after?
Author
Owner

Running go test -count=1 ./...:

Before:

Executed in   16.17 secs    fish           external
   usr time   53.12 secs  988.00 micros   53.12 secs
   sys time    5.41 secs  280.00 micros    5.41 secs

After:

Executed in   11.54 secs    fish           external
   usr time   52.61 secs  897.00 micros   52.61 secs
   sys time    5.19 secs  388.00 micros    5.19 secs

So, a 30% improvement on my machine.

Running `go test -count=1 ./...`: Before: ``` Executed in 16.17 secs fish external usr time 53.12 secs 988.00 micros 53.12 secs sys time 5.41 secs 280.00 micros 5.41 secs ``` After: ``` Executed in 11.54 secs fish external usr time 52.61 secs 897.00 micros 52.61 secs sys time 5.19 secs 388.00 micros 5.19 secs ``` So, a 30% improvement on my machine.
dstepanov-yadro approved these changes 2023-05-11 07:05:17 +00:00
Member

What does "TODO: metabase" mean in the PR message?

What does "TODO: metabase" mean in the PR message?
Author
Owner

@ale64bit will do later in a separate PR.

@ale64bit will do later in a separate PR.
acid-ant approved these changes 2023-05-12 09:44:35 +00:00
fyrchik merged commit c62025c836 into master 2023-05-12 09:45:05 +00:00
fyrchik deleted branch parallelize-tests 2023-05-12 09:45:06 +00:00
Author
Owner

Wait, actually, I have already done metabase tests it in the last commit.

Wait, actually, I have already done metabase tests it in the last commit.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#328
No description provided.