search

phuslu / log

Fastest structured logging
MIT License
672 stars 44 forks source link

AsyncWriter should supports writev #12

Closed phuslu closed 4 years ago

phuslu commented 4 years ago

Two step,

  1. add writev support in FileWriter on linux.
  2. add writev in AsyncWriter base on step 1

A draft already pushed to branch writev https://github.com/phuslu/log/blob/writev/async_writev.go

phuslu commented 4 years ago

merged. https://github.com/phuslu/log/commit/1e28388f2942755cc5408ff9315567f263b1f769

phuslu commented 4 years ago

no obviously performance improvment, so reverted at https://github.com/phuslu/log/commit/33e4cded6aa6426b0bda5fccee1d97cfa48983b2