Hi, On Mon, 13 Oct 2025 at 11:42, Xuneng Zhou <[email protected]> wrote: > > Here is patch v3. The comments have been added, and the extra scope > ({}) has been removed as suggested.
Thanks, the code looks good to me! The benchmark results are nice. I have a quick question about the setup, if you are benchmarking with the default io_method (worker), then the results might show the combined benefit of async I/O and streaming I/O, rather than just streaming I/O alone. If that is the case, I would suggest also running a benchmark with io_method=sync to isolate the performance impact of streaming I/O. You might get interesting results. -- Regards, Nazir Bilal Yavuz Microsoft
