Results 589 comments of MSP-Greg

> concern: does existing test suite really test the behavior of stats method fully after this PR? As mentioned in PR #3528, until all the stats hash values are unique,...

As @dentarg mentioned. Maybe change `update_worker_count` to `update_thread_pool_min_max`? I've never liked ThreadPool related methods using the term `worker`. Re an example, it's not clear how to use this with several...

@yuki24 Not sure if there was a reason for not doing so, if not, please rebase? I can have a look tomorrow. We may need an 'integration' test for this,...

@yuki24 Good day. By using the following, each request is sent, the response is generated, and the 'client' reads it. Then the next request is sent. So, it's not filling...

@yuki24 and @nateberkopec I pushed the test fix, CI is passing. Question, and this pertains to the PR, and maybe Puma: What happens if a `min` value is set higher...

Just rebased (thru PR #3462)

@schneems I'll close. Yes, this is still something I intend to finish, but this will/should never be merged.

Below is hey data, similar to data shown in #3487. All connections are run for 4 seconds, either 0.2 * 20 or 0.01 * 400. Each data group shows latency...

@schneems Interesting. On my system (i9 ten core) I have: ```text Puma starting in single mode... * Puma version: 6.4.3 (ruby 3.4.0-p-1) ("The Eagle of Durango") * Min threads: 3...

@schneems Re why you're seeing different results than I am, I'm not sure. But, given the number of threads/processes that are being spun up, maybe a core problem? I've got...