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

Increase the coverage of the apiId cache #10866

Closed
jaehong-kim opened this issue Apr 9, 2024 · 1 comment
Closed

Increase the coverage of the apiId cache #10866

jaehong-kim opened this issue Apr 9, 2024 · 1 comment
Assignees
@jaehong-kim
Labels
enhancement module:agent module:plugin
Milestone
3.0.0

Comments

@jaehong-kim
Copy link
Contributor

Increase the coverage of the apiId cache.

The Pinpoint agent uses the cache without sending method information (className + methodName + descriptor) to the collector every time.
If the information is already stored in the cache when the method calls, only int-type cache values (apiId) can be passed to the collector to reduce the amount of transmission.

Increase the cache size a little more and register additional frequently used methods.
@jaehong-kim jaehong-kim added this to the 3.0.0 milestone Apr 9, 2024
@jaehong-kim jaehong-kim self-assigned this Apr 9, 2024
jaehong-kim added a commit to jaehong-kim/pinpoint that referenced this issue Apr 9, 2024
@jaehong-kim
[pinpoint-apm#10866] Increase the coverage of the apiId cache
2da5bbb
@jaehong-kim jaehong-kim mentioned this issue Apr 9, 2024
Merged
jaehong-kim added a commit that referenced this issue Apr 11, 2024
@jaehong-kim
[#10866] Increase the coverage of the apiId cache
c4d8677
@jaehong-kim
Copy link
Contributor Author

Applied to the 2.5.4-SNAPSHOT version.

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

@jaehong-kim jaehong-kim

Labels
enhancement module:agent module:plugin
Projects
None yet
Milestone
3.0.0
Development

No branches or pull requests
1 participant
@jaehong-kim