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

Full Threading Support #79

Open
stugol opened this issue Apr 9, 2016 · 2 comments
Open

Full Threading Support #79

stugol opened this issue Apr 9, 2016 · 2 comments

Comments

@stugol
Copy link
Contributor

stugol commented Apr 9, 2016

[Placeholder issue to ensure that threading support remains an active high-priority goal]

Crystal doesn't support threading. This is a mistake. Onyx needs to deviate from Crystal in that respect, for both performance reasons and to solve concurrency issues that stem from [a] fibers being sub-optimal in some cases and [b] IO.select being horribly broken for use in fibers.

@ozra
Copy link
Owner

ozra commented Apr 9, 2016

Agreed.

By the time the core of Onyx is implemented and stabilized, it might very well have landed in Crystal, but if not, it will be a high-prio issue for me to take on, since I need it myself. And if deviation is necessary (PR's not accepted in Crystal repo) then that's the only way it can go.

I leave the high-pri tag off for now, since the core is the top-pri until done.

@ozra
Copy link
Owner

ozra commented Apr 22, 2016

I took the liberty to edit out "HIGH PRIORITY: " from title - prioritization is in Roadmap issue.

@ozra ozra changed the title HIGH PRIORITY: Full Threading Support Full Threading Support Apr 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants