-
Notifications
You must be signed in to change notification settings - Fork 865
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cache the pid and hostname globally, flatstr before writing #60
Conversation
Just need to update readme benchmarks?, |
Yours come out nicer (damn maxbook pro 15'') |
added benchmarks, we actually beat bole in every benchmark - but only just in some cases |
@@ -72,52 +72,61 @@ This produces: | |||
<a name="benchmarks"></a> | |||
## Benchmarks | |||
|
|||
As far as we know, it is one of the fastest loggers in town: | |||
As far as we know Pino is fastest logger in town. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would leave this as before. The difference is so tiny, and the approach between bole and pino is essentially the same now.
20ms over 10000 log lines is hardly perceivable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This put us on par with bole for basic and object benchmarks.
Inspired by https://github.com/rvagg/bole/blob/f3ba893c7f948f459615007767e2ffee519bc14c/bole.js#L13-L14.
^^ @rvagg good job!