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

Golang v1.16 Upgrade #3570

Closed
5 tasks
gmalouf opened this issue Feb 4, 2022 · 1 comment
Closed
5 tasks

Golang v1.16 Upgrade #3570

gmalouf opened this issue Feb 4, 2022 · 1 comment
Labels
new-feature-request Feature request that needs triage Team Carbon-11

Comments

@gmalouf
Copy link
Contributor

gmalouf commented Feb 4, 2022

Problem

We recently prepared a release for and then reverted support for Go 1.16 due to an observed performance regression along with concerns around this issue. Our current version, Go 1.14.7 is on a line that no longer receives maintenance updates and misses out on valuable language enhancements as well.

Solution

Review our software in:

  • go-algorand
  • go-algorand-ci
  • Indexer
  • Algonet
  • Sandbox

ensuring we can (and then removing) our usage of nanosecond level timing calls. Upon addressing this, run performance benchmarks against the latest stable Go 1.16 release to understand any potential performance regressions.

At this point, assuming minimal performance impact, we should move forward with the upgrade (working w/ Hyper Flow to update build pipelines, etc).

Dependencies

Carbon-11 resources are focused on critical efforts at the moment, but

Urgency

We'd like to address this by late Q1 or early Q2, as there additional platforms we'd like to support that require Go 1.16 as their minimal version.

@winder
Copy link
Contributor

winder commented Jun 30, 2022

This is done.

@winder winder closed this as completed Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature-request Feature request that needs triage Team Carbon-11
Projects
None yet
Development

No branches or pull requests

2 participants