apache/dubbo

JVMUtil performance improvement

Open

#13.327 geöffnet am 7. Nov. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Java (26.453 Forks)batch import
component/sdkhelp wantedtype/enhancement

Repository-Metriken

Stars
 (41.524 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 7T 14h) (20 gemergte PRs in 30 T)

Beschreibung

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等）。

Contributor Guide

Research-Richtung
Ändern Sie die interne Methode getThreadDumpString so, dass sie OutputStream für die Streaming Ausgabe anstelle von String Verkettung verwendet. Bewerten Sie, ob die Stack Tiefenkontrolle (Standard 32) nach der Umstellung auf Streaming noch ausreichend Leistungsvorteile bietet oder entfernt werden kann.
Tech Stack
java
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaStream API
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70