Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Temporal for orchestration #58

Closed
wants to merge 3 commits into from
Closed

Use Temporal for orchestration #58

wants to merge 3 commits into from

Conversation

hellais
Copy link
Member

@hellais hellais commented Mar 6, 2024

This implements one of the options outlined inside of: #46

What I like about it:

  • It doesn't require a lot of boilerplate and results in a more compact system
  • It's a well thought out system with strict guarantees
  • The people behind it created uber/cadence
  • It has an SDK for go and javascript too
  • It's quite minimal and unopinionated
  • The web UI looks slick and is nice to work with
  • The documentation looks good
  • It has a cloud option

What I don't like:

  • The execution model of ensuring workflows are deterministic is sane, but a bit tricky to work with (to get it up and running quickly I kind of cheated and just marked a lot of things as unsafe)
  • Since it's unopinionated it doesn't have as much stuff around the orchestration part or tells you how to do things
  • It's a bit less of a "batteries included" solution compared to other solutions
  • The fact it supports many languages makes me worry if they have the capacity to support all these SDKs. I assume python should be first class support, but I'm not sure of that.
  • The billing of the cloud option is quite confusing

@hellais hellais changed the title Temporal Use Temporal for orchestration Mar 6, 2024
@hellais
Copy link
Member Author

hellais commented Apr 15, 2024

This has been merged as part of #60

@hellais hellais closed this Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant