fix: Raise the read buffer sizes to 128KiB.

google / mtail

extract internal monitoring data from application logs for collection in a timeseries database

Apache License 2.0

3.82k stars 378 forks source link

fix: Raise the read buffer sizes to 128KiB. #869

Closed jaqx0r closed 3 months ago

jaqx0r commented 3 months ago

The pipe buffer size had been chosen to match the system default buffer size on Linux, but increasing all buffer sizes makes throughput faster by reducing the number of syscalls for large reads.

Tested logically by the bandwidth-delay product.

Tested experimentally per the linked bug.

Fixes: #685

github-actions[bot] commented 3 months ago

Unit Test Results

1 files 27 suites 8m 41s :stopwatch: 648 tests 646 :white_check_mark: 2 :zzz: 0 :x: 1 917 runs 1 911 :white_check_mark: 6 :zzz: 0 :x:

Results for commit b03ed379.