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.

Sign up for GitHub

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

Jump to bottom

use go 1.17 #421

Merged
merged 4 commits into from
Sep 14, 2021
Merged

use go 1.17 #421

merged 4 commits into from
Sep 14, 2021

Conversation

jsirianni
Copy link
Member

CI is failing for Windows because we try to install go 1.16 when 1.17 is already installed. We should just use 1.17 anyway.

@jsirianni jsirianni mentioned this pull request Sep 13, 2021
Merged
4 tasks
djaglowski
djaglowski previously approved these changes Sep 14, 2021
View reviewed changes
@djaglowski djaglowski self-requested a review September 14, 2021 13:17
@djaglowski
Copy link
Member

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.2069591 -0.3275318 131.05968 +2.2939148
1 5000 4.3448195 -0.7932606 137.75525 -0.35736084
1 10000 10.000217 -0.17256641 142.05226 -2.7947235
1 50000 46.380165 -8.670456 172.84254 +0.7086487
1 100000 90.50327 -19.15448 236.3843 -12.416214
10 100 1.9482267 +0.2240876 132.96605 -0.42147827
10 500 5.741393 -0.29310083 139.67227 +1.4567566
10 1000 11.448705 -2.15516 146.43817 -0.5098419
10 5000 54.31181 -1.0673866 178.6906 -0.17996216
10 10000 99.09322 -19.759262 215.8653 -21.043915

@jsirianni jsirianni merged commit 77e05b9 into master Sep 14, 2021
@jsirianni jsirianni deleted the fix-ci branch September 14, 2021 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@djaglowski djaglowski djaglowski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jsirianni @djaglowski