Skip to content

[refactor]: use reflect.TypeFor #7850

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.

Sign up for GitHub

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

Jump to bottom
Open
oncecelll wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[refactor]: use reflect.TypeFor #7850

oncecelll wants to merge 1 commit into from
+5 −6

Conversation

@oncecelll
Copy link

@oncecelll oncecelll commented Jan 9, 2026 •
edited
Loading

Which problem is this PR solving?

Description of the changes

Try to use better api reflect.TypeFor instead of reflect.TypeOf when we have known the type.
More info golang/go#60088

How was this change tested?

Checklist

@oncecelll
[refactor]: use reflect.TypeFor
9a6d7b9 
Signed-off-by: oncecelll <oncecell@outlook.com>
@oncecelll oncecelll requested a review from a team as a code owner January 9, 2026 11:52
@oncecelll oncecelll requested a review from pavolloffay January 9, 2026 11:52
@dosubot dosubot bot added the go Pull requests that update go code label Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pavolloffay pavolloffay Awaiting requested review from pavolloffay pavolloffay is a code owner automatically assigned from jaegertracing/jaeger-maintainers

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

go Pull requests that update go code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@oncecelll