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

Scheduler implementation #56

Open
wants to merge 3 commits into
base: master
from
Open

Scheduler implementation #56

wants to merge 3 commits into from

Conversation

@ioquatix
Copy link
Member

ioquatix commented Jan 11, 2020

This is a prototype of the scheduler which uses the interface provided by ruby/ruby#1870

At the present time, the scheduler is an option set of hooks for implementing non-blocking IO. However, in the future, the scheduler might form the core of the reactor, replacing the selector entirely.

ioquatix added 2 commits Jan 11, 2020
.travis.yml Outdated
@@ -28,6 +28,7 @@ matrix:
env: JRUBY_OPTS="--debug -X+O"
- rvm: truffleruby
- rvm: ruby-head
- rvm: git@github.com:ioquatix/ruby#thread-selector

This comment has been minimized.

Copy link
@ioquatix

ioquatix Jan 11, 2020

Author Member

How can I make this work?

This comment has been minimized.

Copy link
@havenwood

havenwood Jan 11, 2020

Member

I think it'd be:

- rvm: ruby-head-scheduler --url git://github.com/ioquatix/ruby.git --branch selector

P.S. - I meant to push a commit to my fork of your brach to test CI, not your branch itself. Reverted. I'm sorry!

This comment has been minimized.

Copy link
@havenwood

havenwood Jan 11, 2020

Member

Nope, I'm wrong. It doesn't work since Travis is appending a --binary flag. https://travis-ci.org/havenwood/async/jobs/635745650

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 11, 2020

Coverage Status

Coverage decreased (-6.006%) to 83.501% when pulling 6d4ab47 on scheduler into 213f840 on master.

@ioquatix ioquatix force-pushed the scheduler branch from 5ccf14f to 6d4ab47 Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.