grafana / grafana/pyroscope-java

Merge 2 profiler schedulers

Open
#185 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
126
Forks
48
Avg merge
4d 14h
Merged PRs (30d)
5

Description

I suggest to officially support sampling, remove experimental tags, and merge the 2 classes: [ContinuousProfilingScheduler](https://github.com/grafana/pyroscope-java/blob/main/agent/src/main/java/io/pyroscope/javaagent/impl/ContinuousProfilingScheduler.java) and [SamplingProfilingScheduler](https://github.com/grafana/pyroscope-java/blob/main/agent/src/main/java/io/pyroscope/javaagent/impl/SamplingProfilingScheduler.java). Let me know if it makes sense to you and should I send a PR.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.