Skip to content

add benchmark test for FromIncomingContext and ValueFromIncomingContext#6117

Merged
zasweq merged 3 commits intogrpc:masterfrom
wenxuwan:add_benchmark
Mar 15, 2023
Merged

add benchmark test for FromIncomingContext and ValueFromIncomingContext#6117
zasweq merged 3 commits intogrpc:masterfrom
wenxuwan:add_benchmark

Conversation

@wenxuwan
Copy link
Copy Markdown
Contributor

@wenxuwan wenxuwan commented Mar 13, 2023 •
edited by easwars
Loading

add benchmarktest for FromIncomingContext and ValueFromIncomingContext, and the result is:

goos: darwin
goarch: amd64
pkg: google.golang.org/grpc/metadata
cpu: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
BenchmarkFromIncomingContext-12                       4657538               260.4 ns/op           416 B/op          3 allocs/op
BenchmarkValueFromIncomingContext-12            28756804                38.28 ns/op           16 B/op          1 allocs/op
PASS
ok      google.golang.org/grpc/metadata 3.475s

RELEASE NOTES: none

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Mar 13, 2023 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@wenxuwan wenxuwan changed the title add benchmarktest for FromIncomingContext and ValueFromIncomingContext add benchmark test for FromIncomingContext and ValueFromIncomingContext Mar 13, 2023
easwars
easwars reviewed Mar 14, 2023
View reviewed changes
Comment thread metadata/metadata_test.go Outdated
Comment thread metadata/metadata_test.go
@easwars easwars added this to the 1.54 Release milestone Mar 15, 2023
zasweq
zasweq approved these changes Mar 15, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@zasweq zasweq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@zasweq zasweq merged commit 6f44ae8 into grpc:master Mar 15, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@easwars easwars easwars approved these changes

+1 more reviewer

@zasweq zasweq zasweq approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@wenxuwan wenxuwan

Labels

Type: Testing

Projects

None yet

Milestone

1.54 Release

Development

Successfully merging this pull request may close these issues.

3 participants

@wenxuwan @easwars @zasweq