Scope question wrt channels and a "servlet style" setup.. We have a system that creates instance of ...

# coroutines

[taer]

Scope question wrt channels and a "servlet style" setup.. We have a system that creates instance of our class(think servlet) and has lifecycles for that instance. In the "do get" method, I create a produce channel(Bar), and then proceed to consume from it. I might swap to flows, but in this case, is this all setup right wrt the structured concurrency?

When I close the second instance, the channels seem to shutdown correctly. The only thing that feels like it's missing is a server level context(that would parent the instanceScope)

Untitled

[taer]

the spawn call in the system I'm converting is expected to block untill the work is done. Side question though. What couroutingScope is the this inside that runBlocking? The

for (x in streamer)

call is a suspending call, so I'm curious what scope is managing that call

[araqnid]

runBlocking with no context parameter will create a new top-level Job iirc

[taer]

yeah.. I validated that just now in debugger.

[araqnid]

It looks like you should do runBlocking(instanceScope) { … }

[araqnid]

uh, runBlocking(instanceScope.context)

[taer]

is that kosher w/ all the parent things? 🙂 I just read a bit in this channel about messing up the whole heirarchy sometimes.

[araqnid]

although tbh I’d just have an instanceJob : Job and pass context + instanceJob as the parameter to runBlocking

[taer]

ah.. so my close() cancels the instanceJob instead

[taer]

I see it both ways.. "dont manage jobs, that's the scope's job" vs this method

[araqnid]

I think it’s all ok, a coroutine running with a top-level Job by accident is one of the pitfalls though

[taer]

ah that CoroutineScope is just context + Job()

[araqnid]

so where is the Job getting created at all? I thought CoroutineScope(..) didn’t create one, as opposed to coroutineScope { }

[araqnid]

oh it does?

[taer]

ContextScope(if (context[Job] != null) context else context + Job())

[taer]

The names are blurring now.. CoroutineScope, CoroutineContext.

[taer]

It's at least workin. I'm gonna have to reread the hierarchy medium article 3 more times. 🙂

[araqnid]

ah, it creates a Job only if there wasn’t one already

[araqnid]

good luck!

[taer]

thx!