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.

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

Update Go to 1.17 #3533

Closed
ivan-valkov opened this issue Aug 24, 2021 — with Board Genius Sync · 1 comment · Fixed by #3627
Closed

Update Go to 1.17 #3533

ivan-valkov opened this issue Aug 24, 2021 — with Board Genius Sync · 1 comment · Fixed by #3627
Assignees
Labels
triage Needs to be triaged and prioritised accordingly

Comments

Copy link
Contributor

Go 1.17 was released recently. It comes with a number of changes but something that might be of particular interest to us is the "free" ~5% performance improvement we can get. It comes from the new way of passing function arguments and results implemented in the go compiler.

I believe the easiest way to do this will be to bump the go version in the core builder and build a new version.
We can also consider bumping the go version specified in our go.mod files in the executor and operator.

@ivan-valkov ivan-valkov added the triage Needs to be triaged and prioritised accordingly label Aug 24, 2021
@ukclivecox
Copy link
Contributor

ukclivecox commented Sep 1, 2021

we have now moved to go 1.16 so this should be easier to look into.

@ukclivecox ukclivecox removed the triage Needs to be triaged and prioritised accordingly label Sep 2, 2021
@ukclivecox ukclivecox self-assigned this Sep 22, 2021
@axsaucedo axsaucedo added the triage Needs to be triaged and prioritised accordingly label Sep 22, 2021 — with Board Genius Sync
@ukclivecox ukclivecox removed the triage Needs to be triaged and prioritised accordingly label Oct 11, 2021
@seldondev seldondev added the triage Needs to be triaged and prioritised accordingly label Oct 14, 2021 — with Board Genius Sync
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Needs to be triaged and prioritised accordingly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants