grafana / grafana/pyroscope-java
Merge 2 profiler schedulers
- Lingua principale
- Java
- Stelle
- 126
- Fork
- 48
- Merge medio
- 4g 14h
- PR unite (30g)
- 5
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Iniziate confrontando agent/src/main/java/io/pyroscope/javaagent/impl/ContinuousProfilingScheduler.java e SamplingProfilingScheduler.java. Chiarite se il sampling debba essere ufficialmente supportato e cosa dovrebbe comportare la rimozione dei tag sperimentali e l’unione delle classi; nell’issue non sono specificati i criteri di completamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java
- Ambito
- performance
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 20/100