Remove Scala Futures from PlanRunner API

atomist-attic / rug

DEPRECATED Runtime for Rugs

GNU General Public License v3.0

53 stars 13 forks source link

Remove Scala Futures from PlanRunner API #642

Closed cdupuis closed 7 years ago

cdupuis commented 7 years ago

This is to remove runtime complications with using Scala's global ExecutionContext.

Those changes have been tested in rug-runner.

johnsonr commented 7 years ago

Is this all synchronous now?

cdupuis commented 7 years ago

Yes, rug is doing all synchronous now. The concurrency is handled in the runtime via dedicated thread pools. But this behavior can be controlled better now as rug isn't using threading.