Thayne McCombs
Thayne McCombs
hmm, from the failing tests, I suspect we may still need the `ZIO.yieldNow` on Js and Native. Or maybe change the implementation of `WhileLoop` so that it yields between each...
@jdegoes does my diff [here](https://github.com/zio/zio/pull/9131#discussion_r1820325195) look like the right direction? I don't fully understand the run loop, so I'm not entirely sure what changes need to happen.
On a related note, I ran into performance issues when using TSemaphore[1]. My code originally used cats-effects, but for unrelated reasons I was looking to switch to zio. But on...
FWIW, I think https://github.com/zio/zio/pull/9120 should help with the performance, at least in the cases of threads > permits, especially with high contention. I'm not sure why it is slower in...
@rhollander73 would you mind pushing up your changes, or sending a diff?
@debora-ito Is this something worth finishing? Or is it unlikely to get merged?
I've incorporated your patch into my branch
This is probably related to the fact that `stsClient` is a public method inherited from a protected internal class.