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

Task.fork #532

Closed
alexandru opened this Issue Jan 15, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@alexandru
Member

alexandru commented Jan 15, 2018

Repurpose the word to mean Task.start + forced async boundary.

This means introducing this method:

sealed abstract class Task[+A] {
  
  def fork: Task[Task[A]] =
    executeAsync.start
}

But in doing so we need to rename or deprecate older methods and companion object functions:

  • executeWithFork renamed to executeAsync on both Task and Observable
  • Task.fork is now deprecated
  • Observable.fork is now deprecated

Also Task.fromFuture suffered an upgrade, the implementation now avoiding async boundaries when calling onComplete handlers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment