Skip to content
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

Bump benchmarks #1475

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

arukiidou
Copy link
Contributor

@arukiidou arukiidou commented Nov 28, 2024

Benchmark upgrades.

  • Bumps:
    • github.com/rs/zerolog v1.33.0
    • go.uber.org/zap v1.26.0

cpu: Intel(R) Core(TM) i5-10400 CPU @ 2.90GHz
RAM: 64.0 GB (63.8 GB available)

Raw go test output
Running tool: C:\Program Files\Go\bin\go.exe test -benchmem -run=^$ -coverprofile=C:\Users\aruki\AppData\Local\Temp\vscode-go89Cor7\go-code-cover -bench . go.uber.org/zap/benchmarks

goos: windows
goarch: amd64
pkg: go.uber.org/zap/benchmarks
cpu: Intel(R) Core(TM) i5-10400 CPU @ 2.90GHz
BenchmarkDisabledWithoutFields/Zap-12     	991477424	         1.202 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledWithoutFields/Zap.Check-12         	1000000000	         0.8379 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledWithoutFields/Zap.Sugar-12         	100000000	        12.06 ns/op	      16 B/op	       1 allocs/op
BenchmarkDisabledWithoutFields/Zap.SugarFormatting-12         	15014376	        81.01 ns/op	     136 B/op	       6 allocs/op
BenchmarkDisabledWithoutFields/apex/log-12                    	1000000000	         0.6539 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledWithoutFields/sirupsen/logrus-12             	100000000	        11.19 ns/op	      16 B/op	       1 allocs/op
BenchmarkDisabledWithoutFields/rs/zerolog-12                  	1000000000	         0.5698 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledWithoutFields/slog-12                        	983090836	         1.212 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledWithoutFields/slog.LogAttrs-12               	961005595	         1.210 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAccumulatedContext/Zap-12                    	1000000000	         1.205 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAccumulatedContext/Zap.Check-12              	1000000000	         0.8506 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAccumulatedContext/Zap.Sugar-12              	100000000	        11.94 ns/op	      16 B/op	       1 allocs/op
BenchmarkDisabledAccumulatedContext/Zap.SugarFormatting-12    	14125573	        81.53 ns/op	     136 B/op	       6 allocs/op
BenchmarkDisabledAccumulatedContext/apex/log-12               	1000000000	         0.3767 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAccumulatedContext/sirupsen/logrus-12        	100000000	        11.15 ns/op	      16 B/op	       1 allocs/op
BenchmarkDisabledAccumulatedContext/rs/zerolog-12             	1000000000	         0.5729 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAccumulatedContext/slog-12                   	1000000000	         1.161 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAccumulatedContext/slog.LogAttrs-12          	1000000000	         1.186 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAddingFields/Zap-12                          	 4655335	       269.9 ns/op	     800 B/op	       5 allocs/op
BenchmarkDisabledAddingFields/Zap.Check-12                    	1000000000	         0.8770 ns/op	       0 B/op	       0 allocs/op
BenchmarkDisabledAddingFields/Zap.Sugar-12                    	13465126	        89.91 ns/op	     136 B/op	       6 allocs/op
BenchmarkDisabledAddingFields/apex/log-12                     	 3179898	       390.7 ns/op	     886 B/op	      10 allocs/op
BenchmarkDisabledAddingFields/sirupsen/logrus-12              	 1578231	       746.4 ns/op	    1525 B/op	      12 allocs/op
BenchmarkDisabledAddingFields/rs/zerolog-12                   	62479498	        20.77 ns/op	      24 B/op	       1 allocs/op
BenchmarkDisabledAddingFields/slog-12                         	12275712	        92.04 ns/op	     136 B/op	       6 allocs/op
BenchmarkDisabledAddingFields/slog.LogAttrs-12                	 5498755	       217.2 ns/op	     512 B/op	       5 allocs/op
BenchmarkWithoutFields/Zap-12                                 	17354180	        65.16 ns/op	       0 B/op	       0 allocs/op
BenchmarkWithoutFields/Zap.Check-12                           	18802440	        64.56 ns/op	       0 B/op	       0 allocs/op
BenchmarkWithoutFields/Zap.CheckSampled-12                    	50521209	        22.99 ns/op	       0 B/op	       0 allocs/op
BenchmarkWithoutFields/Zap.Sugar-12                           	13166701	        94.53 ns/op	      16 B/op	       1 allocs/op
BenchmarkWithoutFields/Zap.SugarFormatting-12                 	  379504	      3115 ns/op	    1922 B/op	      58 allocs/op
BenchmarkWithoutFields/apex/log-12                            	  915994	      1250 ns/op	     216 B/op	       4 allocs/op
BenchmarkWithoutFields/go-kit/kit/log-12                      	 4085536	       317.3 ns/op	     480 B/op	       8 allocs/op
BenchmarkWithoutFields/inconshreveable/log15-12               	  340838	      3349 ns/op	    1267 B/op	      17 allocs/op
BenchmarkWithoutFields/sirupsen/logrus-12                     	  456068	      2688 ns/op	     940 B/op	      20 allocs/op
BenchmarkWithoutFields/stdlib.Println-12                      	 7231245	       174.6 ns/op	      16 B/op	       1 allocs/op
BenchmarkWithoutFields/stdlib.Printf-12                       	  531598	      2647 ns/op	    1275 B/op	      57 allocs/op
BenchmarkWithoutFields/rs/zerolog-12                          	33290013	        36.34 ns/op	       0 B/op	       0 allocs/op
BenchmarkWithoutFields/rs/zerolog.Formatting-12               	  413586	      3099 ns/op	    1916 B/op	      58 allocs/op
BenchmarkWithoutFields/rs/zerolog.Check-12                    	33634077	        37.30 ns/op	       0 B/op	       0 allocs/op
BenchmarkWithoutFields/slog-12                                	 6424154	       177.1 ns/op	       0 B/op	       0 allocs/op
BenchmarkWithoutFields/slog.LogAttrs-12                       	 6997002	       202.9 ns/op	       0 B/op	       0 allocs/op
BenchmarkAccumulatedContext/Zap-12                            	15914274	        79.21 ns/op	       0 B/op	       0 allocs/op
BenchmarkAccumulatedContext/Zap.Check-12                      	16890987	        76.58 ns/op	       0 B/op	       0 allocs/op
BenchmarkAccumulatedContext/Zap.CheckSampled-12               	42719067	        24.25 ns/op	       0 B/op	       0 allocs/op
BenchmarkAccumulatedContext/Zap.Sugar-12                      	10599498	       107.0 ns/op	      16 B/op	       1 allocs/op
BenchmarkAccumulatedContext/Zap.SugarFormatting-12            	  359053	      3368 ns/op	    1927 B/op	      58 allocs/op
BenchmarkAccumulatedContext/apex/log-12                       	   66946	     16704 ns/op	    2947 B/op	      50 allocs/op
BenchmarkAccumulatedContext/go-kit/kit/log-12                 	  318583	      4273 ns/op	    3353 B/op	      53 allocs/op
BenchmarkAccumulatedContext/inconshreveable/log15-12          	   70899	     16213 ns/op	    3035 B/op	      67 allocs/op
BenchmarkAccumulatedContext/sirupsen/logrus-12                	   60229	     19116 ns/op	    4289 B/op	      65 allocs/op
BenchmarkAccumulatedContext/rs/zerolog-12                     	34217181	        37.22 ns/op	       0 B/op	       0 allocs/op
BenchmarkAccumulatedContext/rs/zerolog.Check-12               	29205393	        41.40 ns/op	       0 B/op	       0 allocs/op
BenchmarkAccumulatedContext/rs/zerolog.Formatting-12          	  404304	      3242 ns/op	    1916 B/op	      58 allocs/op
BenchmarkAccumulatedContext/slog-12                           	 6710350	       188.9 ns/op	       0 B/op	       0 allocs/op
BenchmarkAccumulatedContext/slog.LogAttrs-12                  	 6555466	       183.0 ns/op	       0 B/op	       0 allocs/op
BenchmarkAddingFields/Zap-12                                  	 1000000	      1303 ns/op	     804 B/op	       5 allocs/op
BenchmarkAddingFields/Zap.Check-12                            	  984696	      1360 ns/op	     805 B/op	       5 allocs/op
BenchmarkAddingFields/Zap.CheckSampled-12                     	 6098875	       173.7 ns/op	      91 B/op	       0 allocs/op
BenchmarkAddingFields/Zap.Sugar-12                            	  760252	      1744 ns/op	    1628 B/op	      10 allocs/op
BenchmarkAddingFields/apex/log-12                             	   64658	     17901 ns/op	    3842 B/op	      60 allocs/op
BenchmarkAddingFields/go-kit/kit/log-12                       	  340164	      3946 ns/op	    2958 B/op	      54 allocs/op
BenchmarkAddingFields/inconshreveable/log15-12                	   52942	     21042 ns/op	    6286 B/op	      71 allocs/op
BenchmarkAddingFields/sirupsen/logrus-12                      	   55362	     21871 ns/op	    5836 B/op	      76 allocs/op
BenchmarkAddingFields/rs/zerolog-12                           	 2357716	       641.5 ns/op	      24 B/op	       1 allocs/op
BenchmarkAddingFields/rs/zerolog.Check-12                     	 1752664	       653.7 ns/op	      24 B/op	       1 allocs/op
BenchmarkAddingFields/slog-12                                 	  251642	      4338 ns/op	    3179 B/op	      41 allocs/op
BenchmarkAddingFields/slog.LogAttrs-12                        	  267128	      4526 ns/op	    3559 B/op	      40 allocs/op
PASS
coverage: [no statements]
ok  	go.uber.org/zap/benchmarks	89.484s

Signed-off-by: junya koyama <[email protected]>
@abhinav
Copy link
Collaborator

abhinav commented Dec 7, 2024

@arukiidou Thanks for the PR. This appears to be just a go mod tidy. Did you intend to also update the README template?

@arukiidou
Copy link
Contributor Author

@arukiidou Thanks for the PR. This appears to be just a go mod tidy. Did you intend to also update the README template?

yes, not yet update README.

Signed-off-by: junya koyama <[email protected]>
Signed-off-by: junya koyama <[email protected]>
@arukiidou
Copy link
Contributor Author

@abhinav
I had updating README via:

cat .readme.tmpl | go run internal/readme/readme.go > README.md

@arukiidou arukiidou marked this pull request as ready for review December 19, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants