Skip to content
This repository has been archived by the owner on Jun 19, 2022. It is now read-only.

Logging full event on ingress error causing unnecessary memory usage #1548

Closed
yolocs opened this issue Aug 4, 2020 · 0 comments · Fixed by #1549
Closed

Logging full event on ingress error causing unnecessary memory usage #1548

yolocs opened this issue Aug 4, 2020 · 0 comments · Fixed by #1549
Assignees
@yolocs
Labels
area/broker kind/bug Something isn't working release/1
Milestone
v0.17.0-M2

Comments

@yolocs
Copy link
Member

yolocs commented Aug 4, 2020

From profiling data.

Current

hDPobM0jin8

After removing logging

bsHmaF8wEnn
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@yolocs yolocs

Labels
area/broker kind/bug Something isn't working release/1
Projects
None yet
Milestone
v0.17.0-M2
Development

Successfully merging a pull request may close this issue.

Remove logging full event on publish errors yolocs/knative-gcp
1 participant
@yolocs