danielgmyers / flux-swf-client

Apache License 2.0
10 stars 3 forks source link

Allow loops in graphs! #134

Closed timgalvin closed 3 months ago

timgalvin commented 3 months ago

😜

danielgmyers commented 3 months ago

Thanks Tim ;)

As Tim is aware, because we discussed it before he posted this issue, I have no plans to add support for loops in Flux absent a compelling use case that can't be easily solved with just running the workflow multiple times, or doing the loop inside the implementation of a single step.

For anyone wondering, the main reasons are: 1) The 25k event limit in SWF means loops can't have too many iterations, so I want to encourage design patterns that avoid this limit. (This is a problem for partitioned steps with many partitions, too.) 2) I designed Flux to help users construct workflows that are easy to understand. Loops make the graph significantly more complicated, especially when you add branches into the mix.

If anyone has a compelling use case for loops, please feel free to reopen this issue with details!