Conversation

Keruspe

Based on #866

This uses async-executor under the hood and is highly inspired by both smol and the async-executor implementation inside of async-std.

Pros:

  • less code inside async-std, simpler integration
  • shares the same executors with any other users of async-global-executor
  • allows spawning local tasks on all of the global executor threads

Cons:

  • adds one more extra dependency
  • ???

@yoshuawuyts

Hi, thanks for this PR!

I def see how sharing the global executor between projects could be useful. But in order to accept this PR I'd like to see:

  1. async-global-executor dual licensed as MIT / Apache-2.0. If ever need to migrate away from async-global-executor we shouldn't need to worry about licensing questions.
  2. Granting commit/publish access to @dignifiedquire, @stjepang or myself. Or moving the project to the async-rs org. That way if something ever needs fixing that is time-critical and for some reason we can't reach you, we're still able to move ahead.

I hope that makes sense. Thanks heaps!

@Keruspe

Done and Done.

I'm fine with moving it to async-rs too if you see fit

@KeruspeKeruspe force-pushed the async-global-executor branch from a158cdc to ebd0050 Compare September 7, 2020 15:07
@Keruspe

Rebased on top of the 1.0 bumps

@dignifiedquire

Thanks. I was out for the last two weeks, will be back in the next days and take a look at this.

@KeruspeKeruspe force-pushed the async-global-executor branch from ebd0050 to 03cd32e Compare September 14, 2020 13:01
@Keruspe

Rebased

@Keruspe

CI failure should now be fixed with async-executor 1.1.1

Signed-off-by: Marc-Antoine Perennou <[email protected]>
@KeruspeKeruspe force-pushed the async-global-executor branch from 98f95f9 to d8b0786 Compare September 15, 2020 07:03

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me, given that this is pretty much still the same code, but in a different place for now. I do like having it extracted as its own crate.
Thanks

@dignifiedquiredignifiedquire changed the title RFC: switch to async-global-executor feat: move executor to async-global-executo Sep 15, 2020
@dignifiedquiredignifiedquire merged commit 352c54b into async-rs:master Sep 15, 2020
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.