Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Move GC type into AgentInfo #1532

Closed
Xylus opened this issue Feb 5, 2016 · 0 comments · Fixed by #1538
Closed

Move GC type into AgentInfo #1532

Xylus opened this issue Feb 5, 2016 · 0 comments · Fixed by #1538
Assignees
@Xylus
Labels
enhancement
Milestone
1.5.2

Comments

@Xylus
Copy link
Contributor

Xylus commented Feb 5, 2016

JVM GC type follows the same lifecycle as AgentInfo, and it's access pattern matches it.
Moving JVM GC type out of AgentStat and into AgentInfo would make accessing easier, and would be better for storage efficiency.
@Xylus Xylus added this to the 1.5.2 milestone Feb 5, 2016
@Xylus Xylus self-assigned this Feb 11, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Feb 16, 2016
@Xylus
pinpoint-apm#1532 Move GC type into AgentInfo
dbee9eb
@Xylus Xylus mentioned this issue Feb 16, 2016
Merged
dawidmalina pushed a commit to dawidmalina/pinpoint that referenced this issue Mar 20, 2016
@Xylus @dawidmalina
pinpoint-apm#1532 Move GC type into AgentInfo
8bf9a4a
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@Xylus Xylus

Labels
enhancement
Projects
None yet
Milestone
1.5.2
Development

Successfully merging a pull request may close this issue.

Move GC type into AgentInfo Xylus/pinpoint
1 participant
@Xylus