Does it make sense to add the `flowOn` here: ```in...

# coroutines

[dave08]

Does it make sense to add the flowOn here:

inline fun <reified T : Any, reified R : Any> Flow<T>.concurrentMap(
	dispatcher: CoroutineDispatcher = <http://Dispatchers.IO|Dispatchers.IO>,
	crossinline mapFun: suspend (T) -> R
)Flow<R> = flatMapMerge {
	flow { emit(mapFun(it)) }.flowOn(dispatcher)
}

?

[Zach Klippenstein (he/him) [MOD]]

Looks fine to me

[dave08]

I was thinking that flatMapMerge already uses some dispatcher for concurrency, so that this might be unnecessary?

[Zach Klippenstein (he/him) [MOD]]

All coroutines use a dispatcher, this is useful if you want to map to happen on a different dispatcher (which makes sense, since it’s common for flows to be collected on Main or Default).