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

reduce unnecessary logging #2549

Merged
merged 1 commit into from
Feb 25, 2022
Merged

reduce unnecessary logging #2549

merged 1 commit into from
Feb 25, 2022

Conversation

yiminc
Copy link
Member

@yiminc yiminc commented Feb 25, 2022

What changed?
Reduce unnecessary logging

Why?
Too many logging pollute the log and make it harder to use.

How did you test it?

Potential risks

Is hotfix candidate?

@yiminc yiminc requested a review from a team as a code owner February 25, 2022 20:23
@yiminc yiminc enabled auto-merge (squash) February 25, 2022 20:40
@yiminc yiminc disabled auto-merge February 25, 2022 20:42
@yiminc yiminc merged commit 6107df5 into temporalio:master Feb 25, 2022
yiminc added a commit that referenced this pull request Feb 25, 2022
yux0 pushed a commit that referenced this pull request Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants