mbraceproject / MBrace.Core

MBrace Core Libraries & Runtime Foundations
http://mbrace.io/
Apache License 2.0
211 stars 46 forks source link

CloudFlow.collect requires an explicit seq #63

Closed isaacabraham closed 9 years ago

isaacabraham commented 9 years ago

The signature of CloudFlow.Collect looks like: -

val inline collect : f:('T -> seq<'R>) -> flow:CloudFlow<'T> -> CloudFlow<'R>

Even if you return e.g. an array<'R>, the compiler will complain about this and you need to e.g. Seq.ofArray at the end of the lambda. Could the signature be changed to something like this?

val inline collect : f:('T -> #seq<'R>) -> flow:CloudFlow<'T> -> CloudFlow<'R>

palladin commented 9 years ago

Thanks for the feedback! Fixed in https://github.com/mbraceproject/MBrace.Core/commit/0078c1d048606ae8e17c4f995aed8af87e77a8ab

isaacabraham commented 9 years ago

@palladin Sweet, thanks.