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

Parallelism #69

Closed
drujensen opened this issue Sep 4, 2017 · 8 comments
Closed

Parallelism #69

drujensen opened this issue Sep 4, 2017 · 8 comments

Comments

@drujensen
Copy link
Contributor

drujensen commented Sep 4, 2017

Some of these languages provide parallelism and others don't.

For Crystal, if you have a multicore system, you need to run multiple processes (1 per core) with reuse port enabled to take advantage of them.

@tbrand
Copy link
Collaborator

tbrand commented Sep 4, 2017

I think the parallelism is a language level feature.
Reusing port and running multiple processes on it is a OS level feature.

If crystal supports the parallelism in language level, I will tune each framework.

@OvermindDL1
Copy link
Collaborator

OvermindDL1 commented Sep 5, 2017

For Crystal, if you have a multicore system, you need to run multiple processes (1 per core) with reuse port enabled to take advantage of them.

Wait what why? Does crystal really not have threading?! o.O
That would explain why it does so much worse than even elixir on my multicore server tests...

@drujensen
Copy link
Contributor Author

Not yet. It's planned for 1.0 but for now, you can use reuse port flag.

@AlexWayfer
Copy link
Contributor

I'll try to make Puma use all CPU cores for Ruby frameworks.

@OvermindDL1
Copy link
Collaborator

@AlexWayfer Ping me when a PR is submitted and I'll run the tests. :-)

@waghanza
Copy link
Collaborator

as in #239 @OvermindDL1 noticed SO_REUSEPORT SHOULD be used

@waghanza
Copy link
Collaborator

@drujensen @tbrand Parallelism has been introduced in crystal
https://crystal-lang.org/2019/09/06/parallelism-in-crystal.html

It will be a pleasure to enable this feature in here

@OvermindDL1
Copy link
Collaborator

Ooo nice! That was one huge feature that crystal was lacking!

@waghanza waghanza closed this as completed Jan 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants