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

trace chain element consumes too many resources #1667

Open
NikitaSkrynnik opened this issue Sep 17, 2024 · 1 comment
Open

trace chain element consumes too many resources #1667

NikitaSkrynnik opened this issue Sep 17, 2024 · 1 comment
Assignees

Comments

@NikitaSkrynnik
Copy link
Contributor

Main Issue

#1666

Description

context.WithValue is used in many places (especially in trace chain element). It makes ctx very heavy and significantly slows down some operations like postpone which are also used in many chain elements. As a result forwarder can spend several seconds only on postone.

Solution

Make contexts light and use connection's metadata instead.

@denis-tingaikin
Copy link
Member

left side with enabled trace chain element , right side with disabled trace chain element

memory:
image

cpu:
image

@denis-tingaikin denis-tingaikin moved this to In Progress in Release v1.15.0 Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

No branches or pull requests

2 participants