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

Change tree type in router struct #88

Merged
merged 3 commits into from
Jan 12, 2023

Conversation

bmf-san
Copy link
Owner

@bmf-san bmf-san commented Jan 12, 2023

Description

see: #84

Changes

changes tree type in router struct.

benchmark tests

goos: darwin
goarch: arm64
pkg: github.com/bmf-san/goblin
BenchmarkSetRoutes1  	1000000000	         0.0000022 ns/op	       0 B/op	       0 allocs/op
BenchmarkSetRoutes5  	1000000000	         0.0000018 ns/op	       0 B/op	       0 allocs/op
BenchmarkSetRoutes10 	1000000000	         0.0000025 ns/op	       0 B/op	       0 allocs/op
BenchmarkStatic1     	18641005	        57.88 ns/op	       0 B/op	       0 allocs/op
BenchmarkStatic5     	 6875048	       176.5 ns/op	       0 B/op	       0 allocs/op
BenchmarkStatic10    	 3714705	       359.5 ns/op	       0 B/op	       0 allocs/op
BenchmarkWildCard1   	 5812250	       204.0 ns/op	     328 B/op	       3 allocs/op
BenchmarkWildCard5   	 2605432	       467.2 ns/op	     408 B/op	       3 allocs/op
BenchmarkWildCard10  	 1501742	       853.4 ns/op	     609 B/op	       3 allocs/op
BenchmarkRegexp1     	 3698962	       338.7 ns/op	     336 B/op	       4 allocs/op
BenchmarkRegexp5     	 1000000	      1122 ns/op	     460 B/op	       8 allocs/op
BenchmarkRegexp10    	  647394	      1910 ns/op	     694 B/op	      13 allocs/op
PASS
ok  	github.com/bmf-san/goblin	13.507s

Impact range

No specification changes

Operational Requirements

N/A

Related Issue

#84

Supplement

N/A

@bmf-san bmf-san added the enhancement New feature or request label Jan 12, 2023
@bmf-san bmf-san self-assigned this Jan 12, 2023
@bmf-san bmf-san force-pushed the feature/change-tree-type-in-router-struct branch 2 times, most recently from 40cde34 to 6f667a9 Compare January 12, 2023 17:31
@bmf-san bmf-san force-pushed the feature/change-tree-type-in-router-struct branch from 6f667a9 to bc24e90 Compare January 12, 2023 17:32
@bmf-san bmf-san merged commit 7358a32 into master Jan 12, 2023
@bmf-san bmf-san deleted the feature/change-tree-type-in-router-struct branch January 12, 2023 17:37
@bmf-san bmf-san mentioned this pull request Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant