apache/dubbo

JVMUtil performance improvement

Open

#13.327 aperta il 7 nov 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Java (26.453 fork)batch import
component/sdkhelp wantedtype/enhancement

Metriche repository

Star
 (41.524 star)
Metriche merge PR
 (Merge medio 7g 14h) (20 PR mergiate in 30 g)

Descrizione

https://github.com/apache/dubbo/blob/3.0/dubbo-common/src/main/java/org/apache/dubbo/common/utils/JVMUtil.java

JVMUtil. jstack方法调用整体使用OutputStream作为输出可提升性能，而内部JVMUtil.getThreadDumpString使用String长字符串拼接反而降低了性能。 建议修改内部JVMUtil.getThreadDumpString方法同样为流式处理，减少字符串拼接的性能开销

同时，getThreadDumpString方法内部，可能还考虑性能影响，增加了栈深度控制（默认32）。 这个深度控制、对性能的提升，可能没有流式处理改造能带来的性能提升高。建议应先修改流式处理以提升性能。是否可经测试评估性能影响，取消栈深度控制。以便dubbo的threaddump能够使用在各类Thread分析工具中（如flamegraph.pl等）。

Guida contributor

Direzione di ricerca
Modificare il metodo interno getThreadDumpString per utilizzare OutputStream per l'output in streaming invece della concatenazione di stringhe. Valutare se il controllo della profondità dello stack (default 32) fornisca ancora un beneficio di performance sufficiente dopo il passaggio allo streaming, o se può essere rimosso.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaStream API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70