search

Azure / azure-functions-durable-js

JavaScript library for using the Durable Functions bindings
https://www.npmjs.com/package/durable-functions
MIT License
131 stars 47 forks source link

Should we continue to support compound tasks with no children? #424

Closed hossam-nasr closed 1 year ago

hossam-nasr commented 1 year ago

Currently, the SDK supports creating compoung Tasks (e.g.., WhenAny/WhenAll tasks) without any children. This support can lead to weird scenarios such as #423. Perhaps we should consider not supporting this scenario anymore in v3 (See discussion: https://github.com/Azure/azure-functions-durable-js/issues/423#issuecomment-1397694184)

davidmrdavid commented 1 year ago

@hossam-nasr: Did we both end up creating the same issue: - https://github.com/Azure/azure-functions-durable-js/issues/423#issuecomment-1397701924

davidmrdavid commented 1 year ago

oh I see, this is for v3 support only. Nevermind.

hossam-nasr commented 1 year ago

Discussed offline. Let's not support this scenario in v3 and throw an error when we find it

hossam-nasr commented 1 year ago

Closed by #463