-
Notifications
You must be signed in to change notification settings - Fork 18
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
Should this be merged to Go core? #1
Comments
I don’t think so, although it looks fast on paper, one of the biggest advantages of go’s native channel is the ability to select across multiple channels and broadcast using the close function |
Select, Close, Broadcast sounds like additional features, possibly implemented without deteriorating performance boost? I am very curious what are the drawbacks of this approach. If there are no clear problems, this can be big thing like Ryu or more recent pdqsort. I think we are standing on something big here. |
cc @alphadose |
I have already opened an issue for it golang/go#52652 |
We can gain even further optimizations by using runtime internal methods like gopark instead of spinning mechanism which is the current implementation thereby ensuring minimum required goroutine utilization without any performance drops |
Hope Go can really support this optimization. It's a certainly great project! |
If this is really fast and without downsides, should this be in Go core?
Maybe folks in go-dev google group will get interest.
The text was updated successfully, but these errors were encountered: