(Go: >> BACK << -|- >> HOME <<)
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.

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

feat(contrib/log/zap): If disable this level of logging entirely, av… #3357

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

feat(contrib/log/zap): If disable this level of logging entirely, av… #3357

wants to merge 1 commit into from

Conversation

harbourlga
Copy link
Contributor
@harbourlga harbourlga commented Jul 3, 2024 •
edited
Loading

Description (what this PR does / why we need it):

  • The zap log plug-in implemented for contrib/log currently filters unnecessary log levels after the log information is formatted, which will cause a lot of additional overhead.
  • Referring to the implementation of SugaredLogger, it will first determine whether the log level meets the current needs before formatting the string.

Which issue(s) this PR fixes (resolves / be part of):

None

Other special notes for the reviewers:

None

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 3, 2024
@harbourlga harbourlga force-pushed the feat/zap-log-levelfiltering branch from 6fb5b8e to 97d04f0 Compare July 3, 2024 06:57
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Jul 3, 2024
@harbourlga harbourlga changed the title chore(contrib/log/zap): If disable this level of logging entirely, av… feat(contrib/log/zap): If disable this level of logging entirely, av… Jul 3, 2024
@harbourlga harbourlga force-pushed the feat/zap-log-levelfiltering branch from 97d04f0 to 982b998 Compare July 3, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@harbourlga