-
Notifications
You must be signed in to change notification settings - Fork 42
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
Better benchmarks #79
Comments
Though I've added some additional benchmarking support, this should be an on-going process. Moving out of 2.6 release. |
Could you also please include a Scala Native comparison with slogging (https://github.com/jokade/slogging)? Thanks! |
@david-bouyssie, I'd be happy to, but this is way down on my priority list. If you'd be interested in contributing a PR I'd absolutely include the results in the next benchmark results. |
Not a high priority neither on my side, it was just a suggestion for your next iteration. |
Scribe wins over Slogging for the name alone. :-p |
Great work with the library. I think the current benchmarks might be a bit outdated. The benchmark numbers would be more complete if scribe was benchmarked against log4j's disruptor-based async logger. |
@jvican that's a great point. Any chance you have any interest in putting together a PR to represent that well? Where possible I prefer not to be the one to implement competitor loggers to avoid being accused of slanting in Scribe's favor. |
1.) Better tooling to expose performance flaws
2.) Additional benchmarks to show more features of Scribe
3.) A library to generate graphical reports on the benchmark results
The text was updated successfully, but these errors were encountered: