Benchmarks are outdated
WillAbides opened this issue · comments
WillAbides commented
I was unable to reproduce the published benchmark results on the current HEAD, but I was able to reproduce them by reverting to an old commit. It appears that performance has degraded since then. From what I can tell, removing the ability to set GOLANG_NUMBER_PARSING = false
had a significant impact.
This gist has more details about my attempts to reproduce the result.
Klaus Post commented
@WillAbides See PR with updates numbers. AMD Ryzen 3950, Go 1.16.2 FWIW