Jean Boussier
Jean Boussier
@ko1 would you be ok with me merging this? Either in the current state or with `lineno` always set to `0`? Ref: https://bugs.ruby-lang.org/issues/18559#note-3
> "Puma should make the new import-map-driven experience in Rails 7 as fast as possible" You mean as a development server, or as a production one? Because I can see...
@brenogazzola there's no more sprockets, that's the point.
> I don't think that's important right now, because as my benchmark shows, HOL-blocking is the bottleneck and not I/O speed. Well, if you serve each individual static file request...
I did a quick proof of concept: https://github.com/puma/puma/pull/2703 The current `read(write())` ``` Thread Stats Avg Stdev Max +/- Stdev Latency 3.37ms 5.89ms 48.28ms 94.46% Req/Sec 0.88k 148.97 1.07k 82.08% Latency...
> `IO.copy_stream` will be a fast path in many cases. Yeah, one big downside though is that it doesn't have a proper timeout API. You can wrap it with `Timeout.timeout`...
Agreed. I may need to do it one class at a time though because this was too big of a change for me. I'll try to get back at this...
This has been split in 3 distinct PRs, one for each type.
The documentation of `IO.pipe` says [" Not available on all platforms"](https://github.com/ruby/ruby/blob/42a677c895f82bcd611db2773fbe68b0558b142d/io.c#L9605) from 2003 However the method seem to [have special code for win32](https://github.com/ruby/ruby/blob/42a677c895f82bcd611db2773fbe68b0558b142d/io.c#L9700) added in 2011. So it's really not...
Fixed in https://github.com/redis-rb/redis-client/pull/65