sjansen

πŸ“… Joined in 2015

πŸ”Ό 405 Karma

✍️ 83 posts

πŸŒ€
15 latest posts

Load

(Replying to PARENT post)

They (re)wrote most of the client SDKs on a Rust core, but the Temporal server is still written in Go.
πŸ‘€sjansenπŸ•‘1yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

The site loads fine for me.

See also: https://downforeveryoneorjustme.com/temporal.io

πŸ‘€sjansenπŸ•‘1yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

Putting work in a queue is only the start. Most organizations start there and gradually write ad hoc logic as they discover problems like dependencies, retries, & scheduling.

Dependencies: what can be done in parallel and what must be done in sequence? For example, three tasks get pushed in the queue and only after all three finish a fourth task must be run.

Retries: The concept is simple. The details are killer. For example, ifa task fails, how long should the delay between retries be? Too short and you create a retry storm. Forget to add some jitter and you get thundering hoards all retrying at the same time.

Scheduling: Because cron is good enough, until it isn't.

A good workflow solution provides battle tested versions of all of the above. Better yet, a great workflow solution makes it easier to keep business logic separate from plumbing so that it's easier to reason about and test.

πŸ‘€sjansenπŸ•‘1yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

Would "we screwed up by blindly trusting ChatGPT" annoy you less? Because that's how I read it.

Or more specially, given the context: "We were in a rush to translate a bunch of code and ChatGPT was doing such an impressive job helping that we became complacent and forgot that it just parrots back text it has seen before with something that looks like intelligence but without actual comprehension. So when it copied a common bug, we weren't paying enough attention to catch it."

πŸ‘€sjansenπŸ•‘1yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

Line 56 is executed as the file is loaded. Simplified, the line is essentially:

  id = Column(default=str(uuid.uuid4()))

As written, a UUID is generated once and used to set a class-level attribute. Each Python process would generate a unique value, so it wouldn't be immediately obvious. Most of the time Python's ability to run code as a file is loaded is helpful, but this is one the well known gotchas.

Although I'm not a SQL Alchemy user, I assume the fix is essentially the same as it would be for Django. So the correct code would have been essentially:

  id = Column(default=uuid.uuid4)

Instead of executing `uuid4()` and caching a single UUID value, it would have executed `uuid4()` each time a new object was created.
πŸ‘€sjansenπŸ•‘1yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

The author defined fair in a pretty industry standard way:

> One user should not be able to block another's work.

A multi-tenant architecture implies over committing resources to achieve better economics. Once upon a time, decades ago, when computers were so expense that companies payed for batch processing instead of owning their own, FIFO was acceptable. (And for a tiny part of the market it still is.) But ever since the rise of time-sharing operating systems, users have come to expect short delays for small tasks. If you let one customer monopolize your capacity because they submitted a mountain of work before a second customer submits a handful, you probably won't have the second customer much longer.

πŸ‘€sjansenπŸ•‘1yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

It actually uses Flex/Bison, and does take advantage of the extended features they provide.
πŸ‘€sjansenπŸ•‘2yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

Was just going to say, they're merely making more obvious a situation that has long existed. But most people don't bother checking if packages they depend on are part of `main` or `universe` so I can see how this comes as a shock to some.
πŸ‘€sjansenπŸ•‘2yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

> Why would you decouple the change from its context?

The issue is the context.

At work, we require a link to the issue/story because it makes SOC2 audits much easier. The commit message is a great place to explain the chosen implementation, but that's not all there is to software development.

It rarely makes sense to repeat in commits how the issue was reported, or why it was given a specific priority. And it can't track anything after merge, like how the change was tested, or whether the change was communicated to users.

Maybe you don't have to deal with auditors. Maybe your work has different processes to keep them happy. But as far as I'm concerned, is it important to link the issue for context? Yes! Just yes.

πŸ‘€sjansenπŸ•‘3yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

Experience has shown that often β€œworse is better”. Go does an amazing job of balancing complexity and power. I haven’t seen a ”better” language that isn’t either slower, harder to become productive, or both.

https://en.wikipedia.org/wiki/Worse_is_better

πŸ‘€sjansenπŸ•‘3yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

In Go, `rune` is an alias for `int32` and is used to indicate the value is a Unicode "code point".

For characters in the ASCII range, that means it's just a character encoded using more bits. If you need to worry about the full Unicode range then it's important to understand Unicode Normalization Forms.

https://go.dev/blog/strings

https://en.wikipedia.org/wiki/Unicode_equivalence

πŸ‘€sjansenπŸ•‘3yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

> Fake crypto is not, as far as I know, a problem.

As it happens, crypto-less cryptocurrency is very much possible as demonstrated by the OneCoin scam: https://en.wikipedia.org/wiki/OneCoin

https://www.fool.com/the-ascent/cryptocurrency/articles/what...

πŸ‘€sjansenπŸ•‘3yπŸ”Ό0πŸ—¨οΈ0

(Replying to PARENT post)

Goramit! You've reminded me Firefly was killed too ruttin quickly.
πŸ‘€sjansenπŸ•‘3yπŸ”Ό0πŸ—¨οΈ0