grafana / grafana/pyroscope-java
Merge 2 profiler schedulers
- Vorherrschende Sprache
- Java
- Sterne
- 126
- Forks
- 48
- Ø Merge
- 4 T. 14 Std.
- Gemergte PRs (30 T.)
- 5
Beschreibung
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.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginnen Sie mit dem Vergleich von agent/src/main/java/io/pyroscope/javaagent/impl/ContinuousProfilingScheduler.java und SamplingProfilingScheduler.java. Klären Sie, ob Sampling offiziell unterstützt werden sollte und was das Entfernen der experimentellen Tags sowie das Zusammenführen der Klassen beinhalten sollte; im Issue sind keine Abschlusskriterien angegeben.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- performance
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 20/100